Skip to content
This repository has been archived by the owner on Jun 5, 2024. It is now read-only.

Manipulate share type for oc10 guests #1046

Merged
merged 3 commits into from
Apr 26, 2022
Merged

Conversation

AlexAndBear
Copy link

No description provided.

@AlexAndBear AlexAndBear changed the title Manipulate share type Manipulate share type for oc10 guests Apr 25, 2022
src/shareInfo.js Outdated Show resolved Hide resolved
Copy link
Contributor

@kulmann kulmann left a 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. 😁

@AlexAndBear
Copy link
Author

Code itself looks good. But the linter is screaming and the changelog is empty. grin

DRAFT DRAFT DRAFT :D :D :D
I even wasn't able to check if it works, as stuff in the other pr is failing :P

@AlexAndBear AlexAndBear force-pushed the manipulate-share-type branch from ce26d64 to 6c9e28a Compare April 25, 2022 12:22
Comment on lines -12 to -13
// Below keys don't need to be stored
const notNeededKeys = ['item_type', 'item_source', 'file_source', 'parent', 'storage', 'mail_send']
Copy link
Author

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

@AlexAndBear AlexAndBear marked this pull request as ready for review April 26, 2022 10:39
@AlexAndBear AlexAndBear force-pushed the manipulate-share-type branch from 919e352 to df86311 Compare April 26, 2022 13:43
Copy link
Contributor

@kulmann kulmann left a 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 👍

src/shareInfo.js Outdated Show resolved Hide resolved
src/shareManagement.js Outdated Show resolved Hide resolved
JanAckermann and others added 2 commits April 26, 2022 16:00
…pplies

Co-authored-by: Benedikt Kulmann <benedikt@kulmann.biz>
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@AlexAndBear AlexAndBear merged commit 77ab4c0 into master Apr 26, 2022
@delete-merged-branch delete-merged-branch bot deleted the manipulate-share-type branch April 26, 2022 14:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants