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.

Sign up for GitHub

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

Jump to bottom

[Cleanup] Remove workaround that only intercepts Pusher + UrbanAirship updates and only applies correct form of reportActions data #14453

Closed
roryabraham opened this issue Jan 21, 2023 · 1 comment
Closed

[Cleanup] Remove workaround that only intercepts Pusher + UrbanAirship updates and only applies correct form of reportActions data #14453

roryabraham opened this issue Jan 21, 2023 · 1 comment
Assignees
@roryabraham
Labels
Engineering Monthly KSv2

Comments

@roryabraham
Copy link
Contributor

HOLD on:

Problem

In #14452 we will be adding a temporary workaround to gracefully migrate reportActions Onyx data.

Solution

Remove this temporary workaround and go back to the client "blindly" applying Onyx updates it receives from the server.
@melvin-bot melvin-bot bot locked and limited conversation to collaborators Jan 21, 2023
@roryabraham roryabraham self-assigned this Jan 25, 2023
@roryabraham
Copy link
Contributor Author

Already reverted

@roryabraham roryabraham changed the title [HOLD] [Cleanup] Remove workaround that only intercepts Pusher + UrbanAirship updates and only applies correct form of reportActions data [Cleanup] Remove workaround that only intercepts Pusher + UrbanAirship updates and only applies correct form of reportActions data Jan 25, 2023
@Expensify Expensify unlocked this conversation Mar 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@roryabraham roryabraham

Labels
Engineering Monthly KSv2
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@roryabraham