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

Refactor update-cache #3309

Open
Macroz opened this issue May 24, 2024 · 0 comments
Open

Refactor update-cache #3309

Macroz opened this issue May 24, 2024 · 0 comments
Labels
Technical Debt Improving internal product quality

Comments

@Macroz
Copy link
Collaborator

Macroz commented May 24, 2024

The rems.db.applications/update-cache function has some room for later refactoring, as it is getting pretty big. maybe we could split it up somehow? one idea would be

[apps change-set] (calculate-updated-apps state ...)
[users change-set] (calculate-updated-users state change-set ...) ; merge change-sets
[roles change-set] (calculate-updated-roles state change-set ...)
new-state (merge apps users roles) ; or (select-keys apps [::raw-apps ::enriched-apps]) etc.

Originally posted by @aatkin in #3305 (comment)

@Macroz Macroz added the Technical Debt Improving internal product quality label May 24, 2024
@Macroz Macroz mentioned this issue May 24, 2024
5 tasks
@meericsc meericsc moved this to Ideas in REMS Task Board May 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Technical Debt Improving internal product quality
Projects
Status: Ideas
Development

No branches or pull requests

1 participant