Skip to content
This repository was archived by the owner on Jul 12, 2023. It is now read-only.

Don't allow longexpires in user report #2370

Merged
merged 2 commits into from
Aug 17, 2022
Merged

Conversation

mikehelmick
Copy link
Contributor

Fixes #2369

Proposed Changes

  • don't allow [longexpires] in the User Report sms template (since it always uses the short expires)
  • add visual feedback to the UI for this.

image

Release Note

Stop allowing `longexpires` in the User Report SMS template since it always uses the `expires` value.

@mikehelmick mikehelmick requested a review from a team as a code owner August 17, 2022 20:34
@mikehelmick mikehelmick enabled auto-merge (squash) August 17, 2022 20:34
@mikehelmick mikehelmick merged commit e31109a into google:main Aug 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SMS Validation and long expires
2 participants