Nextcloud 27 : Replace "Update" with "Edit" in share dialog

[Jerome-Herbinet]

Same as #41680 but specially made for NC 27 because backport failed.

Checklist

• Code is properly formatted
• Sign-off message is added to all commits
• Tests (unit, integration, api and/or acceptance) are included
• Screenshots before/after for front-end changes
• Documentation (manuals or wiki) has been updated or is not required
• Backports requested where applicable (ex: critical bugfixes)

[nfebe]

The original commit has the word update in two places... Does it already exist in once place here?

[Jerome-Herbinet]

The original commit has the word update in two places... Does it already exist in once place here?

@fenn-cs
There is a difference. I noticed that "update" is also here in the file :

server/apps/files_sharing/src/views/SharingDetailsTab.vue

Line 622 in 2f542ed

perms.push('update')

... but in don't think I need or have to change it because it's apparently not a wording call. What do you think ?

[nfebe]

I see where the issue is coming from.... #41301 was never backported so

server/apps/files_sharing/src/views/SharingDetailsTab.vue

Line 643 in 50d6213

[ATOMIC_PERMISSIONS.UPDATE]: this.t('files_sharing', 'Edit'),

Which you updated in master does not exist in stable27.

cc: @Jerome-Herbinet

[nfebe]

@Jerome-Herbinet Can you please rebase against this and add the update? #41736

[Jerome-Herbinet]

I rebased but there are unnecessary commits. I'll fix it tomorrow.

[Jerome-Herbinet]

@fenn-cs it's OK, problems fixed with rebase and I build the assets.
Thanks @zak39 for your precious help once again :-)

[AndyScherzinger]

/compile amend /

[zak39]

/compile amend /

Hi @AndyScherzinger :)

I would like to know : the owner of the PR is able to use the Bot Nextcloud?

I guess your command allows you to build assets and merge with this branch, no?

[AndyScherzinger]

I would like to know : the owner of the PR is able to use the Bot Nextcloud?

@zak39 yes, well anybody can

I guess your command allows you to build assets and merge with this branch, no?

yes, while you can also leave out the amending, that will just amend the compiled assets with the last commit of the PR. So depending on the decision you might not want that.
For details, see https://github.com/nextcloud/server/blob/master/.github/workflows/command-compile.yml

[AndyScherzinger]

/compile amend /

[nfebe]

/compile amend /

[nfebe]

Drone failure unrelated.