4Science / dspace-angular

Angular UI for DSpace and DSpace-CRIS. In the dspace-cris-7 branch you can find the extension to support DSpace-CRIS
BSD 3-Clause "New" or "Revised" License
14 stars 28 forks source link

Fix error message when updating ORCID settings #21

Closed enea4science closed 1 year ago

enea4science commented 1 year ago

References

None

Description

When a researcher tries to update their ORCID settings, an error message (“The update of the synchronization settings failed”) appears. Indeed REST call returned a 200 response code and settings are actually saved.

This PR fixes that by changing the response condition from isSuccess to hasSucceeded as the last one checks for 'success' and 'successStale'.

Checklist

corrad82-4s commented 1 year ago

To be open against DSpace/main if not already done