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.

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

Bug 32282 #127

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Bug 32282 #127

wants to merge 10 commits into from

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