-
Notifications
You must be signed in to change notification settings - Fork 10
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
Using Univ. ID instead of “Library ID” in sul-requests and My Account #3773
Comments
We're 99% sure that 9-digits is correct. Let's move forward. If we get a response from ID card office that 9-digit isn't correct, we can change the validator accordingly. |
Reviewing numbers input in the FOLIO "External System ID", it appears that we have user records with 8, 9 and 10 digits.
We could:
The purpose behind number validation from my POV was to:
Chris may remember other reasons. |
@dbranchini I don't think this discrepancy in the number of digits changes anything in design or instructions for the user. The number printed on the card (and that they will be instructed to use by Library staff) will be the number stored in the FOLIO External System ID field. |
One more thought: I don't know why we're not just validating on the FOLIO field in real time, and then displaying an error if we can't find a match. I suspect we weren't doing that with Symphony because of performance issues, but it's just a guess (and vague memory) |
Connects to sul-dlss/SearchWorks#3773 This commit helps us move the access portfolio beyond using library barcode numbers, leaning instead on the university-assigned and -managed "university ID."
Connects to sul-dlss/SearchWorks#3773 This commit helps us move the access portfolio beyond using library barcode numbers, leaning instead on the university-assigned and -managed "university ID."
Connects to sul-dlss/SearchWorks#3773 This commit helps us move the access portfolio beyond using library barcode numbers, leaning instead on the university-assigned and -managed "university ID."
You're right, @saseestone , the number of digits doesn't change the design or the label, instructions, and/or validation error messaging. If it's possible to validate against the FOLIO field in real-time, that sounds like a better solution to me. |
Connects to sul-dlss/SearchWorks#3773 This commit helps us move the access portfolio beyond using library barcode numbers, leaning instead on the university-assigned and -managed "university ID."
Connects to sul-dlss/SearchWorks#3773 This commit helps us move the access portfolio beyond using library barcode numbers, leaning instead on the university-assigned and -managed "university ID." - [ ] Figure out how to test this and roll it out.
Connects to sul-dlss/SearchWorks#3773 This commit helps us move the access portfolio beyond using library barcode numbers, leaning instead on the university-assigned and -managed "university ID." - [ ] Figure out how to test this and roll it out.
Connects to sul-dlss/SearchWorks#3773 This commit helps us move the access portfolio beyond using library barcode numbers, leaning instead on the university-assigned and -managed "university ID." - [ ] Figure out how to test this and roll it out.
Connects to sul-dlss/SearchWorks#3773 This commit helps us move the access portfolio beyond using library barcode numbers, leaning instead on the university-assigned and -managed "university ID."
Connects to sul-dlss/SearchWorks#3773 This commit helps us move the access portfolio beyond using library barcode numbers, leaning instead on the university-assigned and -managed "university ID."
Noting here that we need to coordinate with Lane Library & the SUL Privileges office before this change is rolled to Production. Lane needs to know because they are using our Requests code in Lane Search. |
Connects to sul-dlss/SearchWorks#3773 This commit helps us move the access portfolio beyond using library barcode numbers, leaning instead on the university-assigned and -managed "university ID."
Connects to sul-dlss/SearchWorks#3773 This commit helps us move the access portfolio beyond using library barcode numbers, leaning instead on the university-assigned and -managed "university ID."
Connects to sul-dlss/SearchWorks#3773 This commit helps us move the access portfolio beyond using library barcode numbers, leaning instead on the university-assigned and -managed "university ID."
Post discussion with @saseestone I'm summarizing the path forward on this work. The stakeholders are not ready for this yet. @cbeer suggested that we leave the backend wired up to accept University ID and continue to accept Library ID (last 10 digits of the barcode). So @mjgiarlo or another developer will change the backend logic to handle accepting both. The UI will revert back to the Library ID label though because no one has the new ID cards yet. For both requests and my account, we should use the following language: Field label: Library ID For requests, the hint will have the above text plus, "It's optional, but may help you track your request in My Library Account." |
Connects to sul-dlss/SearchWorks#3773 This commit helps us move the access portfolio beyond using library barcode numbers, leaning instead on the university-assigned and -managed "university ID."
Connects to sul-dlss/SearchWorks#3773 This commit helps us move the access portfolio beyond using library barcode numbers, leaning instead on the university-assigned and -managed "university ID."
Context
See SearchWorks - Library ID UX document for problem statement, additional information and more context.
Solution
-- https://mylibrary.stanford.edu/login
-- https://requests.stanford.edu/pages/new?item_id=3440309&origin=SAL3&origin_location=SAL3-STACKS - Select option - "I don't have a SUNet ID."
New label:
University ID
New instructions:
This is a 9-digit number that serves as a replacement for the Library ID that was previously utilized.
Validate input (See @saseestone 's comment below.)
Input for ID should validate that the user entered 9 digits.
Message: "Please enter a valid Stanford University ID."
Notes
The text was updated successfully, but these errors were encountered: