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

Use GitHub Pages #33

Merged
merged 2 commits into from
Apr 17, 2024
Merged

Use GitHub Pages #33

merged 2 commits into from
Apr 17, 2024

Conversation

AurevoirXavier
Copy link
Member

@AurevoirXavier AurevoirXavier commented Apr 17, 2024

  1. Use GitHub Pages to host the production release, and reserve Vercel exclusively for previews.
    1. Vercel will be completely removed after Vercel CI won't run for any forked repo #28.
  2. Improve the history feature.
  3. Add gitignore for local development.

Before

https://docs.darwinia.network/msgport/overview/#embracing-the-future
image

After

https://aurevoirxavier.github.io/document/msgport/overview/#embracing-the-future
image

And get a GitHub Pages status preview in the project home page.

image

Copy link
Contributor

github-actions bot commented Apr 17, 2024

@AurevoirXavier AurevoirXavier marked this pull request as draft April 17, 2024 03:15
@AurevoirXavier AurevoirXavier marked this pull request as ready for review April 17, 2024 03:34
@AurevoirXavier AurevoirXavier changed the title Use pages Use GitHub Pages Apr 17, 2024
@fewensa

This comment was marked as outdated.

@hackfisher
Copy link
Contributor

@fewensa

Does it mean you need to check the custom domain again after you update DNS record?

image

Copy link
Contributor

@hackfisher hackfisher left a comment

Choose a reason for hiding this comment

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

give it a try

@hackfisher hackfisher merged commit 123b554 into main Apr 17, 2024
1 check passed
@hackfisher hackfisher deleted the xavier/ci branch April 17, 2024 06:02
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