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

Can't save form when logged-in as admin #186

Closed
LaChope opened this issue Jan 9, 2024 · 2 comments · Fixed by #189
Closed

Can't save form when logged-in as admin #186

LaChope opened this issue Jan 9, 2024 · 2 comments · Fixed by #189
Assignees
@palagdan

Comments

@LaChope
Copy link
Collaborator

LaChope commented Jan 9, 2024 •
edited by blcham
Loading

We would like to show message e.g. "Unable to save record due to the user not being assigned to an institution."

A/C:

  • backend method checks for existence of relation to institution and if it is not there a proper message is shown in frontend
@LaChope
Copy link
Collaborator Author

LaChope commented Jan 9, 2024 •
edited by blcham
Loading

When logged in as admin, it is not possible to save form (see screenshot below):

image

The manifestation of the error is strange as it says that "requiredUniqueNonEmptyLocalName constraint failed", but it is caused by the user (in this case admin) not being assigned to the organization.

@blcham
Copy link

blcham commented Jan 9, 2024 •
edited
Loading

Related to #72

@blcham blcham closed this as completed Jan 9, 2024
@blcham blcham reopened this Apr 14, 2024
@blcham blcham mentioned this issue Apr 14, 2024
Open
@blcham blcham transferred this issue from kbss-cvut/record-manager Jul 17, 2024
palagdan added a commit that referenced this issue Jul 19, 2024
@palagdan
[#186] Updated error msg indicating that user cannot create a record …
5bb2375 
…without assigned institution
@palagdan palagdan mentioned this issue Jul 19, 2024
Merged
palagdan added a commit that referenced this issue Jul 19, 2024
@palagdan
[#186]Fixed bug with endless pending in case of the error
6691135
palagdan added a commit that referenced this issue Jul 19, 2024
@palagdan
[#186] Updated mock get request to check if the user has institution …
bd243ae 
…or not
palagdan added a commit that referenced this issue Jul 19, 2024
@palagdan
[#186] Fixed wrong api call
89ad126
palagdan added a commit that referenced this issue Jul 19, 2024
@palagdan
[#186] Deleted logs in createRecord function
749b4bd
palagdan added a commit that referenced this issue Jul 19, 2024
@palagdan
[#186] Updated url in creation record test
2923bf8
blcham pushed a commit that referenced this issue Jul 20, 2024
@palagdan @blcham
[#186] Updated error msg indicating that user cannot create a record …
09c907d 
…without assigned institution
blcham pushed a commit that referenced this issue Jul 20, 2024
@palagdan @blcham
[#186]Fixed bug with endless pending in case of the error
0264c38
blcham pushed a commit that referenced this issue Jul 20, 2024
@palagdan @blcham
[#186] Updated mock get request to check if the user has institution …
5407153 
…or not
blcham pushed a commit that referenced this issue Jul 20, 2024
@palagdan @blcham
[#186] Fixed wrong api call
e513dad
blcham pushed a commit that referenced this issue Jul 20, 2024
@palagdan @blcham
[#186] Deleted logs in createRecord function
15397e2
blcham pushed a commit that referenced this issue Jul 20, 2024
@palagdan @blcham
[#186] Updated url in creation record test
58b111d
blcham pushed a commit that referenced this issue Jul 21, 2024
@palagdan @blcham
[#186] Updated error msg indicating that user cannot create a record …
6def2b7 
…without assigned institution
blcham pushed a commit that referenced this issue Jul 21, 2024
@palagdan @blcham
[#186]Fixed bug with endless pending in case of the error
ba8b992
blcham pushed a commit that referenced this issue Jul 21, 2024
@palagdan @blcham
[#186] Updated mock get request to check if the user has institution …
87d8faa 
…or not
blcham pushed a commit that referenced this issue Jul 21, 2024
@palagdan @blcham
[#186] Fixed wrong api call
79db504
blcham pushed a commit that referenced this issue Jul 21, 2024
@palagdan @blcham
[#186] Deleted logs in createRecord function
c57f929
blcham pushed a commit that referenced this issue Jul 21, 2024
@palagdan @blcham
[#186] Updated url in creation record test
2aea891
palagdan added a commit to kbss-cvut/record-manager that referenced this issue Jul 22, 2024
@palagdan
[kbss-cvut/record-manager-ui#186] Implement checkUserInstituion handl…
035b48d 
…er for checking if the user is assigned to any institution
palagdan added a commit to kbss-cvut/record-manager that referenced this issue Jul 22, 2024
@palagdan
[kbss-cvut/record-manager-ui#186] Change Path Url
b310bc7
palagdan added a commit to kbss-cvut/record-manager that referenced this issue Jul 22, 2024
@palagdan
[kbss-cvut/record-manager-ui#186] Make tests for handler verifyCurre…
86bd33b 
…ntUserInstitution
@palagdan palagdan mentioned this issue Jul 22, 2024
Merged
palagdan added a commit to kbss-cvut/record-manager that referenced this issue Jul 22, 2024
@palagdan
[kbss-cvut/record-manager-ui#186] Delete verifyCurrentUserInstitutuio…
a2b7cc4 
…n handler and test for it
palagdan added a commit to kbss-cvut/record-manager that referenced this issue Jul 22, 2024
@palagdan
[kbss-cvut/record-manager-ui#186] Change createRecords handler to han…
ead3ae9 
…dle a situation when User is not assigned to any institutuion
palagdan added a commit to kbss-cvut/record-manager that referenced this issue Jul 22, 2024
@palagdan
[kbss-cvut/record-manager-ui#186] Update Status in case if user does …
3102dc4 
…not have any institution BAD_REQUEST -> CONFLICT
palagdan added a commit to kbss-cvut/record-manager that referenced this issue Jul 22, 2024
@palagdan
[kbss-cvut/record-manager-ui#186] Change description of error msg in …
c8189cb 
…createRecord handler
palagdan added a commit that referenced this issue Jul 22, 2024
@palagdan
[#186] Update createRecords
d6ea0eb
blcham pushed a commit to kbss-cvut/record-manager that referenced this issue Jul 22, 2024
@palagdan @blcham
[kbss-cvut/record-manager-ui#186] Improve error message when user is …
fbfae24 
…not assigned to an institution
palagdan added a commit that referenced this issue Jul 22, 2024
@palagdan
[#186] Change a structure of createRecords
d1e0809
blcham pushed a commit that referenced this issue Jul 22, 2024
@palagdan @blcham
[#186] Implement error handling when user has no institution assigned
93ec96d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@palagdan palagdan

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

186 updated error msg kbss-cvut/record-manager-ui
3 participants
@blcham @LaChope @palagdan