Add address to an existing user fails

Description

Overview: When testing https://folio-org.atlassian.net/browse/UIU-136#icft=UIU-136 following, I tested the overall functionality in the Users app, and then I discovered that adding the address to an existing user is failing
 
Steps to repro:
1. Login to FOLIO test
2. Go to Users, and select a given user
3. Click the '+ Add another address' button, and fill in the form with address information, but leave out the Address type.
 
Expected results:
Being able to save the address, when the Address type is not set to be a required data field.
 
Actual results:
Adding the address is failing, and you get error messages. See screen dump a
 
Additional info:
If I only enter e.g. City information, and specify the address type, then the address will be saved, but that is not a proper address. See screen dump b

CSP Request Details

None

CSP Rejection Details

None

Potential Workaround

None

Attachments

2
  • 17 Aug 2017, 01:43 PM
  • 17 Aug 2017, 01:43 PM

Checklist

hide

TestRail: Results

Activity

Show:

Charlotte Whitt August 18, 2017 at 9:43 AM

Manual tested in FOLIO test http://folio-testing.aws.indexdata.com/ version @folio/users 2.9.100469, using Chrome.

If an address is added, but address type is not selected, then the user now gets an error message saying "Address type is required". All looks good, and I'll close the issue as done.

Michal Kuklis August 17, 2017 at 2:59 PM

I fixed it by adding a validation to addressType in STCOM-45

If the idea here is to be able to save the address without the address type present then I think the fix will have to happen on the server and we will need to create another story for it under mod-users project. I'm going to close this one for now.

Done

Details

Assignee

Reporter

Labels

Priority

TestRail: Cases

Open TestRail: Cases

TestRail: Runs

Open TestRail: Runs
Created August 17, 2017 at 1:39 PM
Updated August 18, 2017 at 9:43 AM
Resolved August 18, 2017 at 9:43 AM
TestRail: Cases
TestRail: Runs

Flag notifications