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

fix(files_sharing): hide download permission for circle shares #39146

Merged
merged 1 commit into from Jul 10, 2023

Conversation

st3iny
Copy link
Member

@st3iny st3iny commented Jul 4, 2023

  • Resolves: (not yet reported; tried to search but didn't find anything)

Summary

Toggling the download permission for circle shares is not implemented yet. Changing the permissions and reloading the page will just show it as enabled again.

The download permission option should be hidden until it's properly implemented.

Before After
grafik grafik

Checklist

@st3iny st3iny added the 3. to review Waiting for reviews label Jul 4, 2023
@st3iny st3iny self-assigned this Jul 4, 2023
@st3iny st3iny added this to the Nextcloud 28 milestone Jul 4, 2023
@st3iny st3iny requested review from a team, susnux, Fenn-CS and Pytal and removed request for a team July 4, 2023 13:53
Copy link
Contributor

@susnux susnux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we should create an issue for this so this TODO does not get forgotten?

Copy link
Contributor

@Fenn-CS Fenn-CS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we should create an issue for this so this TODO does not get forgotten?

+1 on that

This looks good! I would add an approval once the issue for todo is created an linked to avoid an accidental merge and the eventual oblivion of the todo

@st3iny
Copy link
Member Author

st3iny commented Jul 5, 2023

Done! I created a feature request issue.

Ref #39161

@st3iny st3iny requested a review from Fenn-CS July 6, 2023 05:42
apps/files_sharing/src/components/SharingEntry.vue Outdated Show resolved Hide resolved
Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>
@st3iny st3iny force-pushed the fix/hide-circle-download-permission branch from 2876541 to 270d3a4 Compare July 10, 2023 06:52
@st3iny st3iny added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jul 10, 2023
@st3iny st3iny enabled auto-merge July 10, 2023 06:52
@st3iny
Copy link
Member Author

st3iny commented Jul 10, 2023

/backport to stable27

@st3iny
Copy link
Member Author

st3iny commented Jul 10, 2023

/backport to stable26

@st3iny
Copy link
Member Author

st3iny commented Jul 10, 2023

/backport to stable25

@st3iny st3iny merged commit ba2e243 into master Jul 10, 2023
38 checks passed
@st3iny st3iny deleted the fix/hide-circle-download-permission branch July 10, 2023 07:28
@backportbot-nextcloud
Copy link

The backport to stable27 failed. Please do this backport manually.

# Switch to the target branch and update it
git checkout stable27
git pull origin stable27

# Create the new backport branch
git checkout -b fix/foo-stable27

# Cherry pick the change from the commit sha1 of the change against the default branch
# This might cause conflicts. Resolve them.
git cherry-pick abc123

# Push the cherry pick commit to the remote repository and open a pull request
git push origin fix/foo-stable27

More info at https://docs.nextcloud.com/server/latest/developer_manual/getting_started/development_process.html#manual-backport

@backportbot-nextcloud
Copy link

The backport to stable26 failed. Please do this backport manually.

# Switch to the target branch and update it
git checkout stable26
git pull origin stable26

# Create the new backport branch
git checkout -b fix/foo-stable26

# Cherry pick the change from the commit sha1 of the change against the default branch
# This might cause conflicts. Resolve them.
git cherry-pick abc123

# Push the cherry pick commit to the remote repository and open a pull request
git push origin fix/foo-stable26

More info at https://docs.nextcloud.com/server/latest/developer_manual/getting_started/development_process.html#manual-backport

@backportbot-nextcloud
Copy link

The backport to stable25 failed. Please do this backport manually.

# Switch to the target branch and update it
git checkout stable25
git pull origin stable25

# Create the new backport branch
git checkout -b fix/foo-stable25

# Cherry pick the change from the commit sha1 of the change against the default branch
# This might cause conflicts. Resolve them.
git cherry-pick abc123

# Push the cherry pick commit to the remote repository and open a pull request
git push origin fix/foo-stable25

More info at https://docs.nextcloud.com/server/latest/developer_manual/getting_started/development_process.html#manual-backport

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish bug feature: circles Related to circles feature: sharing
Projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants