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

fix: #692 Admin FOM should not be editable when commenting open. #693

Conversation

ianliuwk1019
Copy link
Collaborator

@ianliuwk1019 ianliuwk1019 commented Sep 11, 2024

ref: #692
Business requirement changes due to issue the licenses edit FOM during commenting open (description field) and it has legal consequence. New rule: All FOM fields should not be editable except attachments fields during commenting open.

  • Adjust fom-add-edit component so that fields are only new/editable during FOM initial state. On Commenting open and any other state they will be disabled. Some field are only editable during "create" (initial state).

Thanks for the PR!

Any successful deployments (not always required) will be available below.

Once merged, code will be promoted and handed off to following workflow run.


Thanks for the PR!

Any successful deployments (not always required) will be available below.

Once merged, code will be promoted and handed off to following workflow run.

@ianliuwk1019 ianliuwk1019 linked an issue Sep 11, 2024 that may be closed by this pull request
3 tasks
@ianliuwk1019 ianliuwk1019 marked this pull request as ready for review September 11, 2024 21:56
@basilv
Copy link
Collaborator

basilv commented Sep 11, 2024

So my main concern is that we're changing a number of fields to be only editable in initial state when they were editable in multiple states before. So what if these fields still need to be editable when commenting is closed? I'm not saying they need to be, but it isn't clear what change we are actually making, versus just making fields non-editable during commenting-open.

Copy link
Collaborator

@craigyu craigyu left a comment

Choose a reason for hiding this comment

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

I don't have things setup locally to test these but the code looks good!

@craigyu
Copy link
Collaborator

craigyu commented Sep 11, 2024

So my main concern is that we're changing a number of fields to be only editable in initial state when they were editable in multiple states before. So what if these fields still need to be editable when commenting is closed? I'm not saying they need to be, but it isn't clear what change we are actually making, versus just making fields non-editable during commenting-open.

I had a state here to control the inputs depends on whether the form is in "comment open" state
337ee0e

@ianliuwk1019
Copy link
Collaborator Author

So my main concern is that we're changing a number of fields to be only editable in initial state when they were editable in multiple states before. So what if these fields still need to be editable when commenting is closed? I'm not saying they need to be, but it isn't clear what change we are actually making, versus just making fields non-editable during commenting-open.

Thanks Basil! I adjusted the change, some fields are editable during commenting closed according to latest discussion from Julius.

Copy link
Collaborator

@basilv basilv left a comment

Choose a reason for hiding this comment

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

Have some questions...

@ianliuwk1019 ianliuwk1019 merged commit 520e6a3 into main Sep 12, 2024
16 of 17 checks passed
@ianliuwk1019 ianliuwk1019 deleted the hotfix/692-make-fom-description-field-not-editable-when-commenting-period-is-open branch September 12, 2024 15:18
Copy link
Member

@DerekRoberts DerekRoberts left a comment

Choose a reason for hiding this comment

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

One more approval!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Disable FOM form modifications/updates when commenting period is open
4 participants