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

[LIBSEARCH-904] Temporarily make "Text" action inactive and add a message to users #406

Conversation

erinesullivan
Copy link
Contributor

Overview

Due to some Twilio configuration issues, sms messages aren't being delivered to patrons from Alma. They are getting to Twilio and then Twilio is not sending them.

Some background is we needed to have our account associated with a brand, and Twilio went to verify ours on August 28 while umich.edu was down and rejected our request.
This is likely to be the case for 2 - 3 weeks.

The Text action has been disabled until messages are able to be sent again.

This pull request resolves LIBSEARCH-904.

Testing

  • Make sure the PR is consistent in these browsers:
    • Chrome
    • Firefox
    • Safari
    • Edge
  • Run accessibility tests:
    • WAVE
    • ARC Toolkit
    • axe DevTools
  • Go to a record and look at the list of Actions.
    • Is the Text action grayed out?
    • Can you click on it?
    • Is there a message underneath that says Texting records to your mobile device is temporarily unavailable.?
    • Does the message go away when you click on a non-disabled action?
    • Does the message reappear after clicking on the button of an active action?

- Added styling and a message saying that texting records is temporarily unavailable.
@erinesullivan erinesullivan self-assigned this Sep 11, 2023
@erinesullivan erinesullivan merged commit df1fa17 into master Sep 11, 2023
7 checks passed
@erinesullivan erinesullivan deleted the LIBSEARCH-904-temporarily-make-text-action-inactive-and-add-a-message-to-users branch September 11, 2023 20:24
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