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: create specific objects for stash information (DEV-2787) #557
refactor: create specific objects for stash information (DEV-2787) #557
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Really difficult to review. If there was a bug in this PR, I wouldn't find it 🙈 . Is there a way to write tests?
I know, this turned out more complicated than I hoped. But I'm actually rather confident, there were some bugs in it, and all of them were caught by the e2e tests, so I think it should be ok. |
Good to hear, that sounds good. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, but would it be possible to add the permissions?
No description provided.