-
Notifications
You must be signed in to change notification settings - Fork 48
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
Bug fixes, lower thresholds for consumption emissions and enhanced newletter form #506
Conversation
First version of a contribution guide.
…tab (#490) * Improve layout for 404 page * Ensure public assets served from the same domain are opened in a new tab. This helps us only preload Next.js data for internal links leading to actual pages. * Use internal link to the same domain
* add en- and decoding to municipality routing as well as minor cleanup * Update __tests__/utils/generateMunipacitySitemap.test.ts * Update __tests__/utils/generateMunipacitySitemap.test.ts * Update __tests__/utils/generateMunipacitySitemap.test.ts * Update pages/sitemap.xml.ts * Update pages/sitemap.xml.ts * Update utils/generateMunipacitySitemap.ts --------- Co-authored-by: Samuel Plumppu <6125097+Greenheart@users.noreply.github.com>
* lower thresholds for consumption related emissions * update map labels
* Add anchor link to show the newsletter form * Improve newsletter form spacing and use consistent translation keys with namespace included * Add scroll-padding-top to the root html element to allow scrolling to anchor links and still accounting for the fixed header.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
We need the newsletter update with privacy policy etc before merging. Will fix that during the day.
Apart from that, this is ready to publish.
Status on this? |
Reviewed by me alone due to time constraints ahead of the release |
Includes: