Skip to content

Conversation

@auldaj
Copy link

@auldaj auldaj commented Nov 22, 2022

No description provided.

Added page-section to Z39.50 results table.

Testplan
Step 1: Navigate to Cataloguing module and click 'New from Z39.50/SRU' to start a search
Step 2: Carry out a search
Step 3: Confirm that the white background to the table of results is missing
Step 4: Apply patch
Step 6: repeat search
Step 7: Confirm that the white background is now showing.
Fixed capitalisation of ID in User ID

Test plan
Step 1: go to admin>identity providers
Step 2: click 'new identity provider'
Step 3: check map point drop-down
Step 4: verify that User id is written 'User id'
Step 5: apply patch
Step 6: repeat 1-3
Step 7: verify that User ID is written 'User ID'
Added page-section to Z39.50 results table.

Testplan
Step 1: Navigate to Cataloguing module and click 'New from Z39.50/SRU' to start a search
Step 2: Carry out a search
Step 3: Confirm that the white background to the table of results is missing
Step 4: Apply patch
Step 6: repeat search
Step 7: Confirm that the white background is now showing.
Fixed capitalisation of ID in User ID

Test plan
Step 1: go to admin>identity providers
Step 2: click 'new identity provider'
Step 3: check map point drop-down
Step 4: verify that User id is written 'User id'
Step 5: apply patch
Step 6: repeat 1-3
Step 7: verify that User ID is written 'User ID'
Correct capitalisation of opac>OPAC

Test plan
Step 1: Navigate to admin>Identity Providers
Step 2: Add new domain
Step 3: For 'Allow opac' setting, verify that opac is lower case in both label and description
Step 4: Apply patch
Step 5: Navigate to admin>Identity Providers
Step 6: Add new domain
Step 7: For 'Allow OPAC' setting, verify that OPAC is upper case in both label and description
Fixed two further instances of lower case OPAC to OPAC.

Test plan
1. Go to Administration > Additional parameters > Identity providers.
2. Add a 'New identity provider'
3. Once added, select the 'Manage Domains' action for the identity provider.
4. Edit the existing "Identity provider email domains" entry:
5. Note that for the "Identity provider email domains" table, the column heading is 'Allow opac' instead of 'Allow OPAC'.
6. Add a 'New email domain' got the identity provider, note that the label is 'Allow opac' (the hint text is correct).
7. Apply patch
8. Repeat 1-4
9. Note that for the "Identity provider email domains" table, the column heading is now 'Allow OPAC'.
10. Add a 'New email domain' got the identity provider, note that the label is now 'Allow OPAC'
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant