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

Don't use reports for isArchived Check #54590

Open
srikarparsi opened this issue Dec 26, 2024 · 2 comments
Open

Don't use reports for isArchived Check #54590

srikarparsi opened this issue Dec 26, 2024 · 2 comments
Assignees
@srikarparsi @allgandalf
Labels
Daily KSv2

Comments

@srikarparsi
Copy link
Contributor

As a part of the migration to start using the reportNameValuePairs Onyx collection in App, we migrated the private_isArchived key from the reports collection to the reportNameValuePairs collection.

To finish this migration, we need to stop reading from the reports collection here and instead use the reportNameValuePairs collection
@srikarparsi srikarparsi added the Daily KSv2 label Dec 26, 2024
@srikarparsi srikarparsi self-assigned this Dec 26, 2024
@srikarparsi srikarparsi mentioned this issue Dec 26, 2024
Draft
49 tasks
@nkdengineer
Copy link
Contributor

@srikarparsi Can contributor work on this?

@srikarparsi
Copy link
Contributor Author

Hey @nkdengineer, actually have a WIP PR here that I've been working on on the side.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@srikarparsi srikarparsi

@allgandalf allgandalf

Labels
Daily KSv2
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@srikarparsi @allgandalf @nkdengineer