Skip to content

Comments

Recover "WPB-5204 Remove unused APNS_VOIP code (#3695)"#3967

Merged
fisx merged 6 commits intodevelopfrom
wpb6583-clean-up-apns-cruft-in-gundeck
Mar 21, 2024
Merged

Recover "WPB-5204 Remove unused APNS_VOIP code (#3695)"#3967
fisx merged 6 commits intodevelopfrom
wpb6583-clean-up-apns-cruft-in-gundeck

Conversation

@fisx
Copy link
Contributor

@fisx fisx commented Mar 20, 2024

https://wearezeta.atlassian.net/browse/WPB-6583

This reverts commit 133a740 and fixes 66b8c98.

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Mar 20, 2024
@fisx fisx marked this pull request as ready for review March 20, 2024 12:08
@fisx fisx merged commit 232a882 into develop Mar 21, 2024
@fisx fisx deleted the wpb6583-clean-up-apns-cruft-in-gundeck branch March 21, 2024 10:48
akshaymankar added a commit that referenced this pull request May 7, 2024
elland added a commit that referenced this pull request May 8, 2024
* Revert "Recover "WPB-5204 Remove unused APNS_VOIP code (#3695)" (#3967)"

This reverts commit 232a882.

* integration: Refactor push token generation for tests

This will allow writing tests which want more control over the push tokens

* integration: WIP: register push token tests

* WIP: migrating test to /integration

* integration: finished gundeck token test

and delete old test

* [gundeck] Refactored addToken to use ExceptT

* [gundeck] Returns an error when trying to register VOIP tokens.

* format

* changelog

* wire-api/test/unit: Slightly better error message

* Remove ApsPreference

---------

Co-authored-by: Igor Ranieri <igor@elland.me>
@echoes-hq echoes-hq bot added the echoes: technical-roadmap/technical-debt More specific category, to highlight Technical Debt being tackled. label Jul 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

echoes: technical-roadmap/technical-debt More specific category, to highlight Technical Debt being tackled. ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants