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

Frontend implementation of additional feature selection during project creation #1806

Merged
merged 10 commits into from
Sep 24, 2024

Conversation

NSUWAL123
Copy link
Collaborator

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

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

Related Issue

Describe this PR

This PR contains works on additional feature upload on Map Features step.

Screenshots

image_720
image_720
image_360

@github-actions github-actions bot added enhancement New feature or request frontend Related to frontend code labels Sep 24, 2024
@spwoodcock spwoodcock added priority:low Backlog of tasks that will be addressed in time effort:low Likely a few hours labels Sep 24, 2024
@@ -80,7 +80,7 @@ const SelectForm = ({ flag, geojsonFile, customFormFile, setCustomFormFile }) =>
};
useEffect(() => {
if (customFormFile && !customFileValidity) {
dispatch(ValidateCustomForm(`${import.meta.env.VITE_API_URL}/projects/validate-form`, customFormFile));
dispatch(ValidateCustomForm(`${import.meta.env.VITE_API_URL}/projects/validate-form?debug=true`, customFormFile));
Copy link
Member

Choose a reason for hiding this comment

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

Minor change: this shouldn't use debug=true as it doesn't do any actual validation then (it's to return and inspect the form if there are errors).

I will merge anyway then update on the branch πŸ‘

@spwoodcock
Copy link
Member

This was fast! Might as well get it in the release as it doesn't affect any existing functionality πŸ˜„

@spwoodcock spwoodcock merged commit 1819e8c into hotosm:development Sep 24, 2024
5 of 6 checks passed
@spwoodcock spwoodcock changed the title Feat/multi feature selection Frontend implementation of additional feature selection during project creation Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
effort:low Likely a few hours enhancement New feature or request frontend Related to frontend code priority:low Backlog of tasks that will be addressed in time
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants