inveniosoftware / react-invenio-app-ils

Single Page App built with React for InvenioILS.
https://react-invenio-app-ils.readthedocs.io
MIT License
5 stars 19 forks source link

Package: modules version upgrade #497

Closed Pineirin closed 2 years ago

Pineirin commented 2 years ago

Tested the parts of the application that were clearly affected by those modules (both in invenio in cds) with no visible issue.

ntarocco commented 2 years ago

Can this be closed because already included in https://github.com/inveniosoftware/react-invenio-app-ils/pull/498 ?

Pineirin commented 2 years ago

Can this be closed because already included in #498 ?

I already deleted this content in https://github.com/inveniosoftware/react-invenio-app-ils/pull/498 to avoid confusion.

kpsherva commented 2 years ago

can you add just a small description of the tests you performed to see if these new versions are working ?

kpsherva commented 2 years ago

@zzacharo do you know about any possible issues with these versions that we should pay attention while testing ? maybe from RDM...?

Pineirin commented 2 years ago

can you add just a small description of the tests you performed to see if these new versions are working ?

I performed a general testing of the main workflows of the application (such as creating, loans, books, literature requests). For testing "axios" I tested all the search options offered in different parts of the application. For testing "json" I tested various workflows that used that library, such as the bulk extend or the importer tool.