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

A user get an internal servor error as he wants to save changes in his profile #1724

Closed
ManaDeweerdt opened this issue Feb 24, 2021 · 0 comments
Closed

A user get an internal servor error as he wants to save changes in his profile #1724

ManaDeweerdt opened this issue Feb 24, 2021 · 0 comments
Assignees
@jma
Labels
bug Breaks something but is not blocking f: public ui Public interface, as opposed to the professional interface f: user management p-Medium Medium priority (to be solved within 1-2 years)

Comments

@ManaDeweerdt
Copy link
Contributor

Describe the bug

Logged in as a user, if I edit the profile and save, I get an internal servor error.

To Reproduce

Logged in as a user:

  1. Go to RERO ID
  2. Edit the data. Ex: select 'keep history'.
  3. Save
  4. See error

Expected behavior

As a user, I should be able to change my 'keep history' setting, phone number etc.

Context
@ManaDeweerdt ManaDeweerdt added triage bug Breaks something but is not blocking p-Medium Medium priority (to be solved within 1-2 years) labels Feb 24, 2021
@ManaDeweerdt ManaDeweerdt changed the title A user get an internal servor error as he edit his profile and save A user get an internal servor error as he wants to save changes in his profile Feb 24, 2021
@iGormilhit iGormilhit added f: public ui Public interface, as opposed to the professional interface f: user management and removed triage labels Feb 25, 2021
jma added a commit to jma/rero-ils that referenced this issue Mar 1, 2021
@jma
user: add new fields
377ebb6 
* Replaces the phone field by 4 new home, buisness, mobile, other phone
  fields in the user data model.
* Adds country, gender fields in the user data model.
* Adds 3 invenio-userprofiles configuration to specify the list of
  countries, the default country and the read only fields.
* Makes the barcode repetitive.
* Removes all the user from the patron data model.
* Adds the secound address, the local code, the source fields in the
  patron data model.
* Closes rero#1724, rero#1634, rero#1615, rero#1490, rero#1467, rero#1318, rero#1384.

Co-Authored-by: Johnny Mariéthoz <Johnny.Mariethoz@rero.ch>
jma added a commit to jma/rero-ils that referenced this issue Mar 1, 2021
@jma
users: add new fields, move fields from the patron
fdf434b 
* Replaces the phone field by home, business, mobile, other phone fields
  in the user data model.
* Adds country and gender fields in the user data model.
* Adds 3 invenio-userprofiles configuration to specify the list of
  countries, the default country and the read only fields.
* Makes the barcode repetitive.
* Removes all user fields from the patron data model.
* Adds the second address, the local code, the source fields in the
  patron data model.
* Closes rero#1724, rero#1634, rero#1615, rero#1490, rero#1467, rero#1318, rero#1384.

Co-Authored-by: Johnny Mariéthoz <Johnny.Mariethoz@rero.ch>
@jma jma mentioned this issue Mar 1, 2021
Merged
7 tasks
jma added a commit to jma/rero-ils that referenced this issue Mar 1, 2021
@jma
users: add new fields, move fields from the patron
5480e33 
* Replaces the phone field by home, business, mobile, other phone fields
  in the user data model.
* Adds country and gender fields in the user data model.
* Adds 3 invenio-userprofiles configuration to specify the list of
  countries, the default country and the read only fields.
* Makes the barcode repetitive.
* Removes all user fields from the patron data model.
* Adds the second address, the local code, the source fields in the
  patron data model.
* Closes rero#1724, rero#1634, rero#1615, rero#1490, rero#1467, rero#1318, rero#1384, rero#1670.

Co-Authored-by: Johnny Mariéthoz <Johnny.Mariethoz@rero.ch>
jma added a commit to jma/rero-ils that referenced this issue Mar 1, 2021
@jma
users: add new fields, move fields from the patron
3cc4c88 
* Replaces the phone field by home, business, mobile, other phone fields
  in the user data model.
* Adds country and gender fields in the user data model.
* Adds 3 invenio-userprofiles configuration to specify the list of
  countries, the default country and the read only fields.
* Makes the barcode repetitive.
* Removes all user fields from the patron data model.
* Adds the second address, the local code, the source fields in the
  patron data model.
* Closes rero#1724, rero#1634, rero#1615, rero#1490, rero#1467, rero#1318, rero#1384, rero#1670.

Co-Authored-by: Johnny Mariéthoz <Johnny.Mariethoz@rero.ch>
jma added a commit to jma/rero-ils that referenced this issue Mar 1, 2021
@jma
users: add new fields, move fields from the patron
702696c 
* Replaces the phone field by home, business, mobile, other phone fields
  in the user data model.
* Adds country and gender fields in the user data model.
* Adds 3 invenio-userprofiles configuration to specify the list of
  countries, the default country and the read only fields.
* Makes the barcode repetitive.
* Removes all user fields from the patron data model.
* Adds the second address, the local code, the source fields in the
  patron data model.
* Closes rero#1724, rero#1634, rero#1615, rero#1490, rero#1467, rero#1318, rero#1384, rero#1670.

Co-Authored-by: Johnny Mariéthoz <Johnny.Mariethoz@rero.ch>
jma added a commit to jma/rero-ils that referenced this issue Mar 3, 2021
@jma
users: add new fields, move fields from the patron
a9f6dc7 
* Replaces the phone field by home, business, mobile, other phone fields
  in the user data model.
* Adds country and gender fields in the user data model.
* Adds 3 invenio-userprofiles configuration to specify the list of
  countries, the default country and the read only fields.
* Makes the barcode repetitive.
* Removes all user fields from the patron data model.
* Adds the second address, the local code, the source fields in the
  patron data model.
* Closes rero#1724, rero#1634, rero#1615, rero#1490, rero#1467, rero#1318, rero#1384, rero#1670.

Co-Authored-by: Johnny Mariéthoz <Johnny.Mariethoz@rero.ch>
jma added a commit to jma/rero-ils that referenced this issue Mar 3, 2021
@jma
users: add new fields, move fields from the patron
fb8f162 
* Replaces the phone field by home, business, mobile, other phone fields
  in the user data model.
* Adds country and gender fields in the user data model.
* Adds 3 invenio-userprofiles configuration to specify the list of
  countries, the default country and the read only fields.
* Makes the barcode repetitive.
* Removes all user fields from the patron data model.
* Adds the second address, the local code, the source fields in the
  patron data model.
* Closes rero#1724, rero#1634, rero#1615, rero#1490, rero#1467, rero#1318, rero#1384, rero#1670.

Co-Authored-by: Johnny Mariéthoz <Johnny.Mariethoz@rero.ch>
jma added a commit that referenced this issue Mar 3, 2021
@jma
users: add new fields, move fields from the patron
dd8ef3f 
* Replaces the phone field by home, business, mobile, other phone fields
  in the user data model.
* Adds country and gender fields in the user data model.
* Adds 3 invenio-userprofiles configuration to specify the list of
  countries, the default country and the read only fields.
* Makes the barcode repetitive.
* Removes all user fields from the patron data model.
* Adds the second address, the local code, the source fields in the
  patron data model.
* Closes #1724, #1634, #1615, #1490, #1467, #1318, #1384, #1670.

Co-Authored-by: Johnny Mariéthoz <Johnny.Mariethoz@rero.ch>
@jma jma closed this as completed in 30bf6ec Mar 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jma jma

Labels
bug Breaks something but is not blocking f: public ui Public interface, as opposed to the professional interface f: user management p-Medium Medium priority (to be solved within 1-2 years)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jma @iGormilhit @ManaDeweerdt