novu icon indicating copy to clipboard operation
novu copied to clipboard

refactor(app-gen): cache locking

Open djabarovgeorge opened this issue 3 months ago • 3 comments

What changed? Why was the change needed?

This pull request is a work in progress and shouldn't be reviewed yet. It's the first step in a refactoring process, and the next stage will focus on reusing the applyLock method.

I'm also questioning where the locking logic belongs. While I like the cleanness of this solution, I'm unsure if the cache layer is the ideal place for it. Perhaps it should reside within a separate use case. I'd appreciate your thoughts on this matter.

Screenshots

Expand for optional sections

Related enterprise PR

EE

Special notes for your reviewer

djabarovgeorge avatar May 16 '24 12:05 djabarovgeorge