-
Notifications
You must be signed in to change notification settings - Fork 33
Manipulate share type for oc10 guests #1046
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.
Code itself looks good. But the linter is screaming and the changelog is empty. 😁
DRAFT DRAFT DRAFT :D :D :D |
ce26d64
to
6c9e28a
Compare
// Below keys don't need to be stored | ||
const notNeededKeys = ['item_type', 'item_source', 'file_source', 'parent', 'storage', 'mail_send'] |
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.
we need the storage, and I have no idea, why we should purge those properties
919e352
to
df86311
Compare
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.
nitpicks regarding changelog and code comments. the code looks good 👍
changelog/unreleased/enhancement-enforce-share-type-guest-if-applies
Outdated
Show resolved
Hide resolved
…pplies Co-authored-by: Benedikt Kulmann <benedikt@kulmann.biz>
Kudos, SonarCloud Quality Gate passed! |
No description provided.