Skip to content

Conversation

spwoodcock
Copy link
Member

@spwoodcock spwoodcock commented Jul 26, 2025

What type of PR is this? (check all applicable)

  • πŸ• Feature
  • πŸ› Bug Fix
  • πŸ“ Documentation
  • πŸ§‘β€πŸ’» Refactor
  • βœ… Test
  • πŸ€– Build or CI
  • ❓ Other (please specify)

Related Issue

Fixes #2566
Fixes #2514
Fixes #2516
Fixes #2544
Related #2533
Related to #2674

Describe this PR

  • Many reasons for this: mostly about device support.
  • PGLite is still a bit immature for us to use it.
  • Mapping entirely offline wasn't the best idea either - there are better tools available for that like ODK Collect and QField.

Review Guide

Notes for the reviewer. How to test this change?

Checklist before requesting a review

[optional] What gif best describes this PR or how it makes you feel?

@github-actions github-actions bot added bug Something isn't working frontend:mapper Related to the mapper frontend code labels Jul 26, 2025
@spwoodcock spwoodcock requested a review from NSUWAL123 July 26, 2025 23:25
@spwoodcock
Copy link
Member Author

Merging this early to get it tested on stage, with follow up in #2733

@spwoodcock spwoodcock merged commit 0d5e669 into dev Jul 26, 2025
2 checks passed
@spwoodcock spwoodcock deleted the fix/remove-pglite branch July 26, 2025 23:27
@spwoodcock spwoodcock self-assigned this Jul 26, 2025
@NSUWAL123
Copy link
Contributor

@spwoodcock, so we are not having enableWebforms on the config file and only relying on the option provided on the create project to either use webforms or the ODK Collect app? If yes, then I'll make the necessary changes as it's making the webforms always disabled on the mapper frontend.
image

@spwoodcock
Copy link
Member Author

@spwoodcock, so we are not having enableWebforms on the config file and only relying on the option provided on the create project to either use webforms or the ODK Collect app? If yes, then I'll make the necessary changes as it's making the webforms always disabled on the mapper frontend.
image

Yes I think so - we used that feature flag during development, but should probably remove it entirely now (replacing with project specific config as you say)

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

Labels

bug Something isn't working frontend:mapper Related to the mapper frontend code

Projects

None yet

2 participants