Skip to content

Redesign website from the ground up#3

Merged
Copetan merged 14 commits intoOrnitheMC:masterfrom
worldwidepixel:master
Sep 10, 2024
Merged

Redesign website from the ground up#3
Copetan merged 14 commits intoOrnitheMC:masterfrom
worldwidepixel:master

Conversation

@worldwidepixel
Copy link
Copy Markdown
Contributor

Now in astro! New look, same great taste.

astro.config.mjs and CNAME need to be changed to match ornithemc.net.

@worldwidepixel
Copy link
Copy Markdown
Contributor Author

Updated CNAME and astro.config.mjs for deployment on Github Pages.
Pages setup will still have to be changed.

Copy link
Copy Markdown
Member

@thdaele thdaele left a comment

Choose a reason for hiding this comment

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

Looks good to me

Copy link
Copy Markdown
Member

@Copetan Copetan left a comment

Choose a reason for hiding this comment

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

Website looks good to me, but could you please remove the .vscode directory? I don't know exactly what's strictly necessary for the website to run and what else is just editor configuration, but please remove anything else that is just used for the editor too.

@calmilamsy
Copy link
Copy Markdown

I'd say add .vscode to the .gitignore to make sure this doesn't happen again.

@Copetan Copetan requested a review from SRAZKVT September 9, 2024 16:10
@worldwidepixel
Copy link
Copy Markdown
Contributor Author

@Copetan / @calmilamsy .vscode and legacy/ removed.

Copy link
Copy Markdown
Member

@Copetan Copetan left a comment

Choose a reason for hiding this comment

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

everything looks good now

@Copetan Copetan merged commit 8852ecb into OrnitheMC:master Sep 10, 2024
@worldwidepixel
Copy link
Copy Markdown
Contributor Author

🎉

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.

4 participants