Skip to content

[Major]: Update website to new look #28

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

Merged
merged 8 commits into from
May 7, 2024

Conversation

allgandalf
Copy link
Collaborator

@allgandalf allgandalf commented Apr 29, 2024

closes #29

@allgandalf
Copy link
Collaborator Author

I will work in this migration over the week, as you suggested @mdietze , Have put all contents from old website to one single folder,

We need to make some changes to the repo settings, I will update here once that is needed,

Note

Also can I have admin/repo access here ? I think I need it in case we need some quick PR fixes!!

People that I think should keep their 👀 on this @dlebauer @robkooper @infotroph @eshantri

@allgandalf allgandalf marked this pull request as ready for review April 29, 2024 12:27
@allgandalf
Copy link
Collaborator Author

PR up for review

@dlebauer
Copy link
Member

@GandalfGwaihir thank you for all of your hard work on this! Is there a staging site where I can take a look at the changes?

@allgandalf
Copy link
Collaborator Author

@dlebauer , here is the website deployed on /web route:

https://pecanproject.github.io/web/

@allgandalf allgandalf self-assigned this Apr 29, 2024
@dlebauer dlebauer self-requested a review May 7, 2024 17:57
Copy link
Member

@dlebauer dlebauer left a comment

Choose a reason for hiding this comment

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

Although I haven't reviewed all of the (>1000 file and ~30k line changes), I have reviewed the website and in my opinion the new website is ready to release.

@dlebauer dlebauer merged commit 0b2cb0b into PecanProject:master May 7, 2024
@allgandalf

This comment was marked as resolved.

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.

Upgrade website to new look
2 participants