Skip to content

feat: Implement PWA Support for Mobile Installation#118

Open
kalyan-1845 wants to merge 9 commits into
daviddprtma:mainfrom
kalyan-1845:feature/pwa-support
Open

feat: Implement PWA Support for Mobile Installation#118
kalyan-1845 wants to merge 9 commits into
daviddprtma:mainfrom
kalyan-1845:feature/pwa-support

Conversation

@kalyan-1845

Copy link
Copy Markdown
Contributor

Fixes #116

Hi @daviddprtma! Here is the implementation to upgrade AgroDex into a Progressive Web App (PWA) for SSoC'26.

Changes Included:

  1. Integrated \�ite-plugin-pwa\ into the build process.
  2. Configured the App Manifest inside \�ite.config.ts\ (defining the app name, colors, and auto-update behaviors).
  3. The Vite build will now automatically inject Service Workers for asset caching, vastly improving load times and offline accessibility for users on the ground.

Could you please review this when you have some time? If it looks good, I would really appreciate the \SSoC'26\ labels. Thank you!

@vercel

vercel Bot commented Jun 18, 2026

Copy link
Copy Markdown

@kalyan-1845 is attempting to deploy a commit to the daviddprtma's projects Team on Vercel.

A member of the Team first needs to authorize it.

@daviddprtma daviddprtma moved this from Todo to In Progress in AgroDex SSoC'26 Project Jun 19, 2026
Comment thread pnpm-lock.yaml Outdated

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

delete this file as pnpm is no longer needed.

@kalyan-1845

Copy link
Copy Markdown
Contributor Author

Thank you for the review! I have deleted the \pnpm-lock.yaml\ file here as well and switched the dependencies to standard
pm. The updates are pushed and ready for review!

@daviddprtma

Copy link
Copy Markdown
Owner

@kalyan-1845 you need to resolve the merge conflicts because I can't see your work without you need to do that first. thanks.

@kalyan-1845

Copy link
Copy Markdown
Contributor Author

Thank you for pointing that out! I have resolved the merge conflicts with the upstream main branch and pushed the fix. The PR should be clean and mergeable now. Sorry for the inconvenience!

@daviddprtma

Copy link
Copy Markdown
Owner

please delete pnpm-lock.yaml as the file is no longer needed.

Screenshot 2026-06-19 204559

@kalyan-1845

Copy link
Copy Markdown
Contributor Author

Done! I have deleted the pnpm-lock.yaml file from this branch. The commit has been pushed. Thank you for pointing it out!

@kalyan-1845

Copy link
Copy Markdown
Contributor Author

Hi @daviddprtma! I have deleted the pnpm-lock.yaml file as you requested in your last comment. The PR should be clean and ready for review now. Could you please take a look when you have some free time? Thank you so much!

@daviddprtma

Copy link
Copy Markdown
Owner

can you add this code to your vite.config.ts in your plugins?

export default defineConfig({
plugins: [
VitePWA({
// ... your other PWA configurations
workbox: {
maximumFileSizeToCacheInBytes: 7000000 // Set to 7 MB
}
})
]
});

because I want to pass the frontend CI based on the error that I checked in there and the solution is added those code into your vite.config.ts. thanks.

@daviddprtma

Copy link
Copy Markdown
Owner

hello @kalyan-1845 there's a merge conflicts in your PR as today AgroDex have a new changes and you need to solve your merge comflicts first so I can see your work. thanks.

- Resolved merge conflicts with upstream main
- Added workbox.maximumFileSizeToCacheInBytes: 7000000 (7MB) to VitePWA config as requested
@kalyan-1845

Copy link
Copy Markdown
Contributor Author

Hi @daviddprtma! I have made both changes:

  1. Resolved merge conflicts with the latest upstream main
  2. Added workbox config with maximumFileSizeToCacheInBytes: 7000000 (7 MB) to the VitePWA plugin in �ite.config.ts as you requested

The frontend CI should pass cleanly now. Thank you for your guidance!

Comment thread package-lock.json

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

delete this file as it is not needed to push in the commit.

Comment thread fix131.txt

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

delete this file too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

Feature: Implement PWA Support for Mobile Installation

2 participants