-
Notifications
You must be signed in to change notification settings - Fork 0
feat: improve seo and add pwa support #76
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
Conversation
|
It would be nice to merge this before monday 👀 @gxnca @nunom27 @enricoprazeres |
|
Added the install button and updated the prompt logic to only be shown automatically at most 3 times, which basically means that:
And all of this only if the user is always using the browser and not the app, on Android or iOS, to open the platform during the 3 consecutive weeks. I think this is a good balance of giving a little motivation to the users to use the web app but also not annoying them forever if they insist in using it on the browser. |
|
I think the the PWA prompt is a bit too intrusive. If I recall correctly, it would be the first time we do this in any of our apps, right? Maybe let's just keep it as a button like we did with Calendarium? 🤔 |
|
I think the objective of the prompt is to let our users know of a feature they won't even know is there otherwise. Like Calendarium, the UX in Pombo benefits a lot from the extra vertical space the PWA provides when showing events and classes in the calendar, besides it's an overall better experience. I think this prompt would really benefit the adoption rates of this feature, since the concept of a PWA is not commonly known. As for it being too intrusive, in my opinion it is almost as intrusive as a notification would be. As a user, I wouldn't mind being shown this one time, I think I would even think it's a cool thing we have that option. I discussed this yesterday with @nunom27 and @gxnca and we had come to an agreement. But if you guys want to reconsider, it's okay. I've made my point. |
gxnca
left a comment
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.
![]()
|
🙏🏼 😭 portugal, lisboa, conseguimos. |
Since the
export const metadatafeature of Next.js only works server-side, I have placed the metadata for each page in its ownlayout.tsxfile. This way, thepage.tsxfile of each page is free to be used client-side and has the bonus of not getting cluttered with metadata stuff.Added a install prompt with an "add to home screen" button that will:
Example on iOS:
Example on Android:
Screen_Recording_20250908_201928_Chrome.mp4