Skip to content

Configure Survey Layer section - mobile app redesign - #495

Merged
tomasMizera merged 22 commits into
mainfrom
iss_437-mar-attr-forms-widgets
Oct 4, 2024
Merged

tomasMizera merged 22 commits into
mainfrom
iss_437-mar-attr-forms-widgets

Conversation

@alex-cit

@alex-cit alex-cit commented Jul 28, 2024 •

Copy link
Copy Markdown
Contributor

related to #437
related to #422
fix #155

Redirects:
https://merginmaps.com/docs/layer/settingup_forms/ > https://merginmaps.com/docs/layer/form-widgets/
https://merginmaps.com/docs/layer/settingup_forms_settings/ > https://merginmaps.com/docs/layer/form-configuration/
https://merginmaps.com/docs/layer/settingup_forms_photo/ > https://merginmaps.com/docs/layer/photos/
https://merginmaps.com/docs/layer/exif_metadata/ > https://merginmaps.com/docs/layer/exif/
https://merginmaps.com/docs/layer/working_with_nonspatial_data/ > https://merginmaps.com/docs/layer/non-spatial-data/

Redesign:

The whole section Configure Survey Layer should now have screenshots from the redesigned app (and updated projects). Done in a single PR due to the redirects.

@alex-cit alex-cit added the redirects-update We need to update redirects when this change is released label Jul 28, 2024
@alex-cit alex-cit changed the title WIP Attributes form widgets, config, photos - mobile app redesign Configure Survey Layer section - mobile app redesign Sep 8, 2024
@alex-cit
alex-cit requested a review from tomasMizera September 8, 2024 12:00

@tomasMizera tomasMizera left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Great changes! I checked ~30% of texts so far, here are my comments

Comment thread src/layer/form-widgets/index.md
Comment thread src/layer/form-widgets/index.md
Comment thread src/layer/form-widgets/index.md
Comment thread src/layer/form-widgets/index.md Outdated
Comment thread src/layer/form-widgets/index.md Outdated
Comment thread src/layer/form-widgets/index.md

@tomasMizera tomasMizera left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Very good content, I left a bunch of comments, but they are rather small updates. Thanks for this!

Comment thread src/layer/form-widgets/index.md Outdated
Comment thread src/layer/form-widgets/index.md Outdated
Comment thread src/layer/form-widgets/index.md
Comment thread src/layer/form-widgets/index.md Outdated
Comment thread src/layer/form-widgets/index.md Outdated
Comment thread src/layer/photos/index.md
Comment thread src/layer/photos/index.md Outdated
Comment thread src/layer/form-configuration/index.md
Comment thread src/layer/form-configuration/index.md
Comment thread src/layer/form-configuration/index.md Outdated
@alex-cit

Copy link
Copy Markdown
Contributor Author

There are still some comments from the review that will be addressed later, so still work in progress :)

@alex-cit
alex-cit requested a review from tomasMizera October 4, 2024 12:05

@tomasMizera tomasMizera left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🥳

@tomasMizera

Copy link
Copy Markdown
Contributor

CI will work once merged

@tomasMizera
tomasMizera merged commit 9885747 into main Oct 4, 2024
@tomasMizera
tomasMizera deleted the iss_437-mar-attr-forms-widgets branch October 4, 2024 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

redirects-update We need to update redirects when this change is released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Conditional visibility of fields

2 participants