search

inveniosoftware / invenio-rdm-records

DataCite-based data model for InvenioRDM flavour.
https://invenio-rdm-records.readthedocs.io
MIT License
15 stars 84 forks source link

notifications: replace guest access request notification #1454

Closed kpsherva closed 11 months ago

kpsherva commented 1 year ago

https://github.com/inveniosoftware/invenio-rdm-records/blob/master/invenio_rdm_records/requests/access/requests.py#L104 -> replace with notification (invenio-notifications) on accept action

kpsherva commented 11 months ago

While testing the chain of the PRs, I tried to request guest access, resulted in, with no email sent:

Image

Steps to reproduce:

  1. create a record with restricted files
  2. go to "share" on the records details page
  3. Allow requesting access for non authenticated users
  4. save
  5. go to private browser window, paste the URL to the record, try to request access
  6. see backend console

authenticated user access request works fine