search

google / ground-platform

Ground hosted components: Web console, Cloud Functions, db config
http://groundplatform.org
Apache License 2.0
205 stars 84 forks source link

[LOI Editor] Include LOI changes with publish job action #1895

Closed jcqli closed 1 month ago

jcqli commented 3 months ago

When LOIs are imported for a job that is not published, they do not show up in the map as they are not associated with any jobs.

We can bundle the import/delete LOI action within the publish job action:

[ ] Add a state to each LOI and manage the states so that in the edit job phase [ ] user import LOIs, LOIs shows in edit page, user clicks publish button => LOIs shows in map page [ ] user import LOIs, LOIs shows in edit page, user quits editing without clicking publish => LOIs does not show in map page [ ] user delete LOIs, LOIs does not show in edit page, user clicks publish button => LOIs does not show in map page [ ] user delete LOIs, LOIs does not show in edit page, user quits editing without clicking publish => LOIs shows in map page [ ] Same behavior should happen to a new unpublished job

More context: https://github.com/google/ground-platform/issues/1714#issuecomment-2200410314

jcqli commented 1 month ago

Duplicate of https://github.com/google/ground-platform/issues/1964, using that one to track