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

Resources + router restructuring #96

Merged
merged 17 commits into from
Jul 24, 2022
Merged

Resources + router restructuring #96

merged 17 commits into from
Jul 24, 2022

Conversation

ky28059
Copy link
Member

@ky28059 ky28059 commented Jul 12, 2022

This PR is from an idea from Michael that WATT should host something similar to https://paly.win/nytimes/.

the edupass hyperlink is really annoying to find as an individual student so it could be good to provide quick access

If we made more tabs for everything then the utilities section would grow even more crowded and annoying to navigate on mobile, so I thought having articles like how to register for a free nytimes subscription would be better on their own path, and have maybe one tab in utilities as a hub that links to all of them. The design of said article pages is up for critique and improvement (it's current heavily inspired by certain pages on my website).

Also, student resources might be good as an article instead of a utilities tab.

@ky28059 ky28059 requested a review from ImNotRog July 12, 2022 20:37
@github-actions
Copy link

github-actions bot commented Jul 12, 2022

Visit the preview URL for this PR (updated for commit 2780104):

https://gunnwatt--pr96-articles-30bh1eez.web.app

(expires Wed, 27 Jul 2022 23:26:49 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

@ky28059
Copy link
Member Author

ky28059 commented Jul 12, 2022

We could also rename the whole thing to resources if articles is too vague or unrelated.

This was linked to issues Jul 13, 2022
@saumyasinghal747
Copy link
Contributor

@ky28059

We could also rename the whole thing to resources if articles is too vague or unrelated.

Yes, I think this is a good idea.

This was linked to issues Jul 17, 2022
@ky28059 ky28059 linked an issue Jul 18, 2022 that may be closed by this pull request
Also, move resource pages to their own subdirectory of `/components` (from `/components/utilities`)
TODO: consider making a fallback of some sort?
@ky28059 ky28059 changed the title Articles Resources + router restructuring Jul 18, 2022
@ky28059 ky28059 added the client To do with the frontend label Jul 22, 2022
@ky28059 ky28059 mentioned this pull request Jul 22, 2022
3 tasks
@ky28059 ky28059 merged commit e61f4da into main Jul 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
client To do with the frontend
Projects
None yet
2 participants