-
Notifications
You must be signed in to change notification settings - Fork 9
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
Stampy redesign #543
Stampy redesign #543
Conversation
…esign # Conflicts: # package-lock.json # package.json
* use real page ids on landing page
Css adjustments
Sundry fixes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mruwnik / @buddy-web3 could you update README, please? this was not the only change, right? 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
...I checked a few files and I don't have any big objections. The site after redesign was supposed to be more professional than the old one, so I am not sure about releasing before fixing most of the small bugs too and cleaning up code for unused features, updating README, ... but I will leave that up to the core team. I can help after 2024-03-30 to do more polishing even after release I guess.
FTR, https://www.aisafety.com/reading-guide is now updated to use iframe from stampy.ai
and DNS in Cloudflare is updated to point stampy.ai to https://stampy-ui.aprillion.workers.dev/ => migration of embed version is not a blocker any more.
… caused the Link component to be destroyed before the user is redirected to its `to=` value.
Search results not redirecting. FIX: increase the `hide` timer, which…
Handle articles in multiple sections
QA here: https://redesign.aprillion.workers.dev