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: add pwa to projectshut #2044

Merged
merged 14 commits into from
Aug 10, 2023
Merged

feat: add pwa to projectshut #2044

merged 14 commits into from
Aug 10, 2023

Conversation

SubhamB2003
Copy link
Contributor

@SubhamB2003 SubhamB2003 commented Aug 9, 2023

Related Issue

solves #2028

@priyankarpal Can you please review my pr as tomorrow the event ended?

Description

Screenshots

Installed in my desktop
image

@vercel
Copy link

vercel bot commented Aug 9, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
projectshut ✅ Ready (Inspect) Visit Preview Aug 10, 2023 9:12am

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Great job, @SubhamB2003! 🎉 Thank you for submitting your first pull request. Your contribution is valuable and we appreciate your efforts to improve our project.

We will promptly review your changes and offer feedback. Keep up the excellent work! Kindly remember to check our contributing guidelines

@github-actions github-actions bot added the wait for reviewers wait for the maintainer label Aug 9, 2023
@SubhamB2003 SubhamB2003 changed the title Add PWA to ProjectsHut feat: add pwa to projectshut Aug 9, 2023
@priyankarpal
Copy link
Owner

@SubhamB2003 please use pnpm install --no-frozen-lockfile

Copy link
Owner

@priyankarpal priyankarpal left a comment

Choose a reason for hiding this comment

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

create a image folder & put images in there or try to use cdn link

@priyankarpal priyankarpal added the changes-required changes-required label Aug 9, 2023
@SubhamB2003
Copy link
Contributor Author

create a image folder & put images in there or try to use cdn link

Hey, actually the images are needed for manifest logos. that's why I did not use any CDN

@priyankarpal
Copy link
Owner

priyankarpal commented Aug 9, 2023

create a image folder & put images in there or try to use cdn link

Hey, actually the images are needed for manifest logos. that's why I did not use any CDN

ok

also, I think we need to add

 /public/sw.js**
 /public/workbox-*.js**
 /public/worker-*.js**
 /public/sw.js.map**
 /public/workbox-*.js.map**
 /public/worker-*.js.map

to the gitignore

@SubhamB2003
Copy link
Contributor Author

create a image folder & put images in there or try to use cdn link

Hey, actually the images are needed for manifest logos. that's why I did not use any CDN

ok

also, I think we need to add

 /public/sw.js**
 /public/workbox-*.js**
 /public/worker-*.js**
 /public/sw.js.map**
 /public/workbox-*.js.map**
 /public/worker-*.js.map

to the gitignore

Okk I complete the changes and update it

@SubhamB2003
Copy link
Contributor Author

@priyankarpal Done

.gitignore Outdated
Comment on lines 47 to 52
**/public/sw.js
**/public/workbox-*.js
**/public/worker-*.js
**/public/sw.js.map
**/public/workbox-*.js.map
**/public/worker-*.js.map
Copy link
Owner

@priyankarpal priyankarpal Aug 9, 2023

Choose a reason for hiding this comment

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

I forgot to mention to add these files & also, delete these files these are important

sw.js
sw.js.map
workbox-3576cac3.js
workbox-3576cac3.js.map
Suggested change
**/public/sw.js
**/public/workbox-*.js
**/public/worker-*.js
**/public/sw.js.map
**/public/workbox-*.js.map
**/public/worker-*.js.map
sw.js
sw.js.map
workbox-3576cac3.js
workbox-3576cac3.js.map

Copy link
Owner

Choose a reason for hiding this comment

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

@SubhamB2003 please check this

public/manifest.json Outdated Show resolved Hide resolved
@SubhamB2003
Copy link
Contributor Author

@priyankarpal Done.

@priyankarpal priyankarpal added gssoc23 only for Girlscript summer of code level3 high level fix for GSSoC and removed changes-required changes-required labels Aug 10, 2023
.gitignore Outdated
Comment on lines 47 to 52
**/public/sw.js
**/public/workbox-*.js
**/public/worker-*.js
**/public/sw.js.map
**/public/workbox-*.js.map
**/public/worker-*.js.map
Copy link
Owner

Choose a reason for hiding this comment

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

@SubhamB2003 please check this

@SubhamB2003
Copy link
Contributor Author

@priyankarpal can I also removed this?

@priyankarpal
Copy link
Owner

@priyankarpal can I also removed this?

yeah

@SubhamB2003
Copy link
Contributor Author

@priyankarpal Done.

@priyankarpal
Copy link
Owner

@SubhamB2003 in future please try to create separate branch then create PR. Avoid to create PR from main branch

@priyankarpal priyankarpal merged commit 237c0f5 into priyankarpal:main Aug 10, 2023
5 checks passed
@SubhamB2003
Copy link
Contributor Author

@SubhamB2003 in future please try to create separate branch then create PR. Avoid to create PR from main branch

Ok.
Thank you happy to contribute.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gssoc23 only for Girlscript summer of code level3 high level fix for GSSoC wait for reviewers wait for the maintainer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: Add PWA (Progressive web app) in ProjectsHut
2 participants