Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cloudflare CI/CD #443

Closed
wants to merge 7 commits into from
Closed

Cloudflare CI/CD #443

wants to merge 7 commits into from

Conversation

poplexity
Copy link
Member

@poplexity poplexity commented Jul 28, 2023

Tracked via #442

Description

This adds Cloudflare CI/CD but does not add proper SSR

Test scenarios

Ensure previews work properly, test that the /testing path renders from the testing.js function with proper env var being populated

Checklist:

  • Ensure this works, merge to dev then to master
  • Migrate DNS in CloudFlare to point at the new production urls: teloscan-mainnet.pages.dev and teloscan-testnet.pages.dev
  • Remove netlify and any old GitHub integrated cloudflare deployments so we don't get the extra PR comments and deploy previews

@github-actions
Copy link

github-actions bot commented Jul 28, 2023

Pull Request Deploy Preview Details

🔨 Built from SHA: e0b892d
🔍 Mainnet Deploy URL: https://pr-443.teloscan-mainnet.pages.dev
🔍 Testnet Deploy URL: https://pr-443.teloscan-testnet.pages.dev

@telosnetwork telosnetwork deleted a comment from netlify bot Jul 28, 2023
@telosnetwork telosnetwork deleted a comment from netlify bot Jul 28, 2023
ezra-sg
ezra-sg previously approved these changes Jul 28, 2023
Copy link
Contributor

@ezra-sg ezra-sg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💯

@pmjanus pmjanus marked this pull request as draft October 18, 2023 15:58
@pmjanus pmjanus closed this Apr 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants