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

feat: improve PWA configuration #2214

Merged
merged 14 commits into from
Jan 23, 2024
Merged

feat: improve PWA configuration #2214

merged 14 commits into from
Jan 23, 2024

Conversation

mrbpp
Copy link
Contributor

@mrbpp mrbpp commented Jan 22, 2024

mrbpp and others added 11 commits January 9, 2024 17:33
---
PWA Errors:
---
- fix logo size
- add shortcut icon
- add maskable icons
- add desktop and phone screenshots (wide and narrow)
Images for PWA, tested, everything is working fine.
[+] Please put images to lnbits/static/images where other images are.

[+] Please install optipng and run tools/optipng.sh script to optimize PNGs
[+] Please put images to lnbits/static/images where other images are.

[+] Please install optipng and run tools/optipng.sh script to optimize PNGs
Updated screenshots .png for desktop and phone with a classic theme, optimized sizes, reduced file size, and cleaned metadata.
Updated screenshots for desktop and phone with a classic theme, optimized sizes, reduced file size, and cleaned metadata.
…ized sizes, reduced file size, and cleaned metadata. +maskable icons

Updated screenshots for desktop and phone with a classic theme, optimized sizes, reduced file size, and cleaned metadata. +maskable icons
Apologies for the mistake.
…o empty file in the tools directory (maybe because of chmod +x optipng.sh)
@prusnak
Copy link
Collaborator

prusnak commented Jan 22, 2024

black in CI is failing

please run poetry run black . to fix the issue

@mrbpp
Copy link
Contributor Author

mrbpp commented Jan 22, 2024

Done.

All done! ✨ 🍰 ✨

@prusnak prusnak requested a review from dni January 22, 2024 16:00
Copy link

codecov bot commented Jan 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (a3b7c76) 59.38% compared to head (7fee2df) 59.38%.

Additional details and impacted files
@@           Coverage Diff           @@
##              dev    #2214   +/-   ##
=======================================
  Coverage   59.38%   59.38%           
=======================================
  Files          60       60           
  Lines        8925     8925           
=======================================
  Hits         5300     5300           
  Misses       3625     3625           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@prusnak prusnak added this to the 0.12.0 milestone Jan 22, 2024
Copy link
Member

@dni dni left a comment

Choose a reason for hiding this comment

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

perfect!

well done

@dni dni changed the title PWA feat: improve PWA configuration Jan 23, 2024
@dni dni merged commit ebfcc46 into lnbits:dev Jan 23, 2024
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants