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

Revert "Merge pull request #825 from Expensify/cm-add-sms-delivery-command #826

Merged

Conversation

techievivek
Copy link
Contributor

This reverts commit acb6856, reversing changes made to 67c905c.

Fixed Issues

Pure revert since this code in unused.

Tests

  1. What unit/integration tests cover your change? What autoQA tests cover your change?
  2. What tests did you perform that validates your changed worked?
    N/A

QA

  1. What does QA need to do to validate your changes?
  2. What areas to they need to test for regressions?
    N/A

…mmand"

This reverts commit acb6856, reversing
changes made to 67c905c.
@techievivek techievivek self-assigned this Dec 27, 2024
@techievivek techievivek requested a review from a team as a code owner December 27, 2024 06:06
@melvin-bot melvin-bot bot requested review from blimpich and removed request for a team December 27, 2024 06:07
@techievivek
Copy link
Contributor Author

CC @carlosmiceli reverting this since it's unused, I updated web-e to include the method for now https://github.com/Expensify/Web-Expensify/pull/44904

@Beamanator Beamanator merged commit 3aafb41 into main Dec 27, 2024
6 checks passed
@Beamanator Beamanator deleted the techievivek_revert_changes_for_resetsmsdeliveryfailure branch December 27, 2024 08:01
@os-botify
Copy link
Contributor

os-botify bot commented Dec 27, 2024

🚀 Published to npm in 2.0.112 🎉

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.

2 participants