Skip to content

Commit

Permalink
deposit: remove old user email when present
Browse files Browse the repository at this point in the history
* 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.
  • Loading branch information
ntarocco authored and zzacharo committed Nov 13, 2024
1 parent 4d8052a commit ac70a24
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions cds/modules/invenio_deposit/api.py
Original file line number Diff line number Diff line change
Expand Up @@ -188,6 +188,8 @@ def merge_with_published(self):
args = [lca.dumps(), first.dumps(), self.dumps()]
for arg in args:
del arg["$schema"], arg["_deposit"]
# pop optional removed key `current_user_mail` when present
arg.get("_cds", {}).pop("current_user_mail", None)
args.append({})
m = Merger(*args)
try:
Expand Down

0 comments on commit ac70a24

Please sign in to comment.