Skip to content

deposit: remove old user email when present#1976

Merged
zzacharo merged 1 commit intoCERNDocumentServer:mainfrom
ntarocco:fix/remove-user-email
Nov 13, 2024
Merged

deposit: remove old user email when present#1976
zzacharo merged 1 commit intoCERNDocumentServer:mainfrom
ntarocco:fix/remove-user-email

Conversation

@ntarocco
Copy link
Contributor

  • the _cds.current_user_mail has been removed, however it might still be present in some previous records' revisions. This makes the publishing of editing videos fail. When the _cds.current_user_mail is found while publishing, it is ignored.

@ntarocco ntarocco requested a review from zzacharo November 13, 2024 15:47
@ntarocco ntarocco force-pushed the fix/remove-user-email branch from b55d0ff to 094e2ad Compare November 13, 2024 15:48
* the `_cds.current_user_mail` has been removed, however it might still
  be present in some previous records' revisions. This makes the
  publishing of editing videos fail. When the `_cds.current_user_mail`
  is found while publishing, it is ignored.
@ntarocco ntarocco force-pushed the fix/remove-user-email branch from 094e2ad to ea83652 Compare November 13, 2024 15:48
@zzacharo zzacharo merged commit ac70a24 into CERNDocumentServer:main Nov 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments