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

[Configuration] values can not be changed under the Email tab #8890

Closed
ridz1208 opened this issue Sep 20, 2023 · 3 comments · Fixed by #8891
Closed

[Configuration] values can not be changed under the Email tab #8890

ridz1208 opened this issue Sep 20, 2023 · 3 comments · Fixed by #8891
Labels
Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label) Discussion Required PR or issue awaiting the resolution of a discussion between all involved parties Priority: Projects PR or issue is a priority for at least one project and should be a higher priority for LORIS

Comments

@ridz1208
Copy link
Collaborator

Describe the bug
This is happening on CBIGR and other VMS of 24+

If you go to configuration -> Email, we currently have in From and Reply-to values 50 and 51 respectively and updating them does not change the value, they simply revert back

image
@ridz1208 ridz1208 added the Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label) label Sep 20, 2023
@ridz1208
Copy link
Collaborator Author

@paiva It seems like the value is indeed saving in the DB but just not being displayed properly in the module

@ridz1208
Copy link
Collaborator Author

I confirmed its an issue with the datatype being set to email

@ridz1208 ridz1208 added Discussion Required PR or issue awaiting the resolution of a discussion between all involved parties Priority: Projects PR or issue is a priority for at least one project and should be a higher priority for LORIS labels Sep 20, 2023
@paiva
Copy link
Contributor

paiva commented Sep 21, 2023

@ridz1208 thanks for the follow up!

@laemtl laemtl mentioned this issue Sep 21, 2023
Merged
driusan pushed a commit that referenced this issue Sep 21, 2023
@laemtl
[Configuration] Fix values of email fields (#8891)
48d4d0e 
Fixes #8890
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label) Discussion Required PR or issue awaiting the resolution of a discussion between all involved parties Priority: Projects PR or issue is a priority for at least one project and should be a higher priority for LORIS
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Configuration] Fix values of email fields laemtl/Loris
2 participants
@paiva @ridz1208