refactore creating record

tu-graz-library / invenio-pure

MIT License

0 stars 3 forks source link

refactore creating record #6

Closed utnapischtim closed 3 years ago

utnapischtim commented 3 years ago

use internal API aka invenio_records.api.Record

It should be checked where the permission handling comes in

[x] create a record
[ ] apply permissions works out of the box
[ ] could be used over celery or in a cronjob
[x] write a class to encapsulate the communication between the module and the database (after loading the content from the pure api, this class should be used to create/update/delete the record in the database)
[ ] handle the file the same way as the metadata with internal API
[ ] could be used with a not yet implemented sibling datamodel

mb-wali commented 3 years ago

Creating a record - As I was told the previous dev was using the invenioRDM data model based class to create a record for pure, BUT to be sure @bengaal could you please add a permalink to where the current creating a record is taking place. to see what class is being used. Thanks

bengaal commented 3 years ago

Creating a record - As I was told the previous dev was using the invenioRDM data model based class to create a record for pure, BUT to be sure @bengaal could you please add a permalink to where the current creating a record is taking place. to see what class is being used. Thanks

Data for the record is being prepared starting here and then it is POSTed here.

bengaal commented 3 years ago

handle the file the same way as the metadata with internal API - I don't think this will work until Invenio sorts it out. As of now it says the functionality is temporarily disabled until the new API is completely compatible with it.