Closed philprime closed 1 month ago
Attention: Patch coverage is 0%
with 81 lines
in your changes missing coverage. Please review.
Project coverage is 4.02%. Comparing base (
1b9be38
) to head (618fa50
).
Files | Patch % | Lines |
---|---|---|
pages/api/v0.2/messages.ts | 0.00% | 74 Missing :warning: |
pages/api/v0.1/messages.ts | 0.00% | 7 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
This pull request introduces the messages endpoint v0.2 which will replace the current v0.1 used by client libraries.
The main necessity of this pull request is caused due to a DTO decoding issue in https://github.com/kula-app/OnLaunch-iOS-Client, where I forgot to implement backwards compatibility when parsing the
ActionType
.The issue in the client will be fixed with https://github.com/kula-app/OnLaunch-iOS-Client/pull/53 but due to apps shipped with the latest release of the iOS client, we are bumping the API endpoint version too, and filter out any newly introduced action types.
This will also require updates to the Android and Flutter clients, but as new action types require changes in the clients anyways, there won't be an issue until then.