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

Enable group stream boxes when in draft and edit mode #4032

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

frankduncan
Copy link
Contributor

@frankduncan frankduncan commented Jul 4, 2024

Fixes #4031

They are disabled when editing a proposal in general, but in the case of a draft, the users sometimes need to be able to edit them in order to finish an application.

This fix may be incomplete, as it does not allow the group forms to have the full functionality of hiding the followup forms when "No" is selected, and showing them when "Yes" is selected.

@frankduncan frankduncan force-pushed the 4031-editing-group-texts branch from 4cbe72a to 85148d5 Compare July 4, 2024 20:25
@frjo frjo force-pushed the 4031-editing-group-texts branch from 85148d5 to 7b6aca6 Compare November 7, 2024 14:42
@frjo
Copy link
Member

frjo commented Nov 7, 2024

@wes-otf Field groups only work when filling in an application form the first time.

I do not remember why we did it this way.

Do you/OTF see a reson for not having them active all the time (create/draft/edit)?

@wes-otf
Copy link
Contributor

wes-otf commented Nov 26, 2024

sorry, this slipped by me. but not that I'm aware of! If anything it would make make sense to me to have them all around

@frjo frjo force-pushed the 4031-editing-group-texts branch from 7b6aca6 to d1014dd Compare November 26, 2024 16:04
@frjo frjo changed the title Enable group stream boxes when in draft mode Enable group stream boxes when in draft and edit mode Nov 26, 2024
@frjo frjo force-pushed the 4031-editing-group-texts branch from 3ab2b60 to 99eb330 Compare December 18, 2024 12:10
@frjo frjo force-pushed the 4031-editing-group-texts branch from 99eb330 to 6f0bda8 Compare January 14, 2025 08:16
@frjo frjo added Type: Enhancement This is an improvement of an existing thing (not a new thing, which would be a feature). Type: Patch Mini change, used in release drafter labels Jan 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Enhancement This is an improvement of an existing thing (not a new thing, which would be a feature). Type: Patch Mini change, used in release drafter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

When editing a saved draft with a required, unselected group form type, application is unable to be completed
3 participants