Skip to content

V2.0 #6

Open
JewishLewish wants to merge 23 commits into
mainfrom
v2.0
Open

V2.0 #6
JewishLewish wants to merge 23 commits into
mainfrom
v2.0

Conversation

@JewishLewish

@JewishLewish JewishLewish commented Nov 19, 2023

Copy link
Copy Markdown
Contributor

V2.0 Updates:

Add Email System (*.domain.com -> *@gmai.com) ❌
Fix CSRF Warning! State not equal in request and response.
Debug Mode (for config.py) ✅
Better looking admin dashboard (that way users don't need to jump to Cloudflare to modify stuff) ❌
Potentially(?) move away from Cloudflare or make the code be able to create/modify subdomains w/o using a Cloudflare Wrapper API ❌
Make SQL DB more friendly to admins ❌

@netlify

netlify Bot commented Nov 19, 2023

Copy link
Copy Markdown

Deploy Preview for capable-cactus-9ea0d7 ready!

Name Link
🔨 Latest commit ffaa3ae
🔍 Latest deploy log https://app.netlify.com/sites/capable-cactus-9ea0d7/deploys/65724dd7a8e72a00082f0f9f
😎 Deploy Preview https://deploy-preview-6--capable-cactus-9ea0d7.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@JewishLewish

Copy link
Copy Markdown
Contributor Author

Modified the following code:

    def delete(self, identifier):
        url = f"https://api.cloudflare.com/client/v4/zones/{self.ZONE_ID}/dns_records/{identifier}"
        response = requests.delete(url, headers=self.headers)
        
        #Thread(target=CACHE_INSTANCE.get_subdomains(force_refresh=True)).start() <- Not Needed! 
        return response

@JewishLewish JewishLewish added enhancement New feature or request good first issue Good for newcomers labels Nov 19, 2023
@JewishLewish

Copy link
Copy Markdown
Contributor Author

Debug Mode (for config.py) ✅

@JewishLewish

Copy link
Copy Markdown
Contributor Author

For Commit #703ef8f,

This should fix the /edit issue -> #7

@ItsNotAlexy

Copy link
Copy Markdown
Collaborator

Boutta troll this man by merging his v2 draft

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request good first issue Good for newcomers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants