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.

Sign up for GitHub

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

Jump to bottom

Make "name" field of record optional #57

Open
blcham opened this issue Dec 19, 2023 · 3 comments · Fixed by #59
Open

Make "name" field of record optional #57

blcham opened this issue Dec 19, 2023 · 3 comments · Fixed by #59

Comments

@blcham
Copy link

blcham commented Dec 19, 2023

Image
@blcham
Copy link
Author

blcham commented Dec 19, 2023 •
edited
Loading

So to implement, do alternatively:

  • (preferred) make it optional and do not disable Save button + on the list of record new entered name should occur
  • hide the field and do not disable Save button

@kostobog kostobog self-assigned this Dec 19, 2023
kostobog added a commit that referenced this issue Dec 19, 2023
@kostobog
[#57] Make local name field optional
9e8d118
kostobog added a commit that referenced this issue Dec 19, 2023
@kostobog
[#57] Save user provided value in local name field
a7aeced
@kostobog kostobog mentioned this issue Dec 19, 2023
Merged
blcham added a commit that referenced this issue Dec 19, 2023
@blcham
Merge pull request #59 from /issues/57-fix-name-field-in-record-form
7bebb8f 
Issues/57 fix name field in record form
@blcham blcham reopened this Dec 19, 2023
@blcham
Copy link
Author

blcham commented Dec 19, 2023

Notes from @kostobog :
The local name field is now optional but it is still defined in the RequiredAttributes.js file.

@blcham
Copy link
Author

blcham commented Dec 20, 2023

TBD

@blcham blcham added question Further information is requested and removed question Further information is requested labels Dec 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Issues/57 fix name field in record form kbss-cvut/record-manager-ui
2 participants
@blcham @kostobog