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

change restrictions on user report SMS template #1944

Merged
merged 1 commit into from
Mar 24, 2021
Merged

change restrictions on user report SMS template #1944

merged 1 commit into from
Mar 24, 2021

Conversation

mikehelmick
Copy link
Contributor

Towards #1928

Proposed Changes

  • Fix deletion of SMS template bug
  • remove length restriction on user report, allow user report template to use ENX deep links

Release Note

User report can use signed SMS and relax template restrictions.

@mikehelmick mikehelmick requested a review from a team as a code owner March 24, 2021 18:48
@googlebot googlebot added the cla: yes Auto: added by CLA bot when all committers have signed a CLA. label Mar 24, 2021
@mikehelmick mikehelmick enabled auto-merge (squash) March 24, 2021 18:53
@mikehelmick mikehelmick merged commit bb7d04f into google:main Mar 24, 2021
@mikehelmick mikehelmick deleted the urSMSRestrictions branch March 24, 2021 19:08
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@sethvargo sethvargo sethvargo approved these changes

@yuriatgoogle yuriatgoogle Awaiting requested review from yuriatgoogle

Assignees
No one assigned
Labels
cla: yes Auto: added by CLA bot when all committers have signed a CLA.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mikehelmick @sethvargo @googlebot