Skip to content

Make Human Connection a Progressive Web App#1668

Closed
Tirokk wants to merge 3 commits intopr1668basefrom
pr1668head
Closed

Make Human Connection a Progressive Web App#1668
Tirokk wants to merge 3 commits intopr1668basefrom
pr1668head

Conversation

@Tirokk
Copy link
Member

@Tirokk Tirokk commented Oct 5, 2020

roschaefer Authored by roschaefer
Sep 22, 2019
Merged Sep 23, 2019


Following this tutorial here: https://pwa.nuxtjs.org/setup.html

🍰 Pullrequest

To my understanding, this is enough to let chrome users on Android add a "Shortcut" to their Home Screen. If we get a Google Account we can even add it to the Play Store.

Issues

  • None

Todo

  • None

@Tirokk
Copy link
Member Author

Tirokk commented Oct 6, 2020

roschaefer Authored by roschaefer
Sep 23, 2019


Yes, found the lighthouse plugin, too! I wonder how we can integrate that into our CI/CD pipeline on several pages that require authentication

Copy link
Contributor

@Mogge Mogge left a comment

Choose a reason for hiding this comment

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

Not sure exactly how to review this...
I pulled it down and everything seems to work as expected, and it's passing the build
I needed to read more about Progressive Web Apps, and found the Lighthouse tool in the Chrome dev tools... here are the results
On alpha desktop:
Screenshot from 2019-09-23 09-35-18
mobile:
Screenshot from 2019-09-23 09-35-52
On feature branch localhost desktop:
Screenshot from 2019-09-23 09-32-23
mobile:
Screenshot from 2019-09-23 09-37-53

it's not really a fair comparison, because one is in production mode the other not, but I found the tool really interesting, and would love to test this, when it gets deployed to nitro-staging
Also, you can see that in the tests on the pwa branch, it is in fact considered a Progressive Web App, by this tool.
Cool that they have a cli that can be used to automate tests and send the reports to someone to analyze. 😸
Really cool @roschaefer

@Mogge Mogge closed this Oct 8, 2020
@ulfgebhardt ulfgebhardt deleted the pr1668head branch January 7, 2021 08:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants