inspirehep / record-editor

Record editing tool used in http://inspirehep.net
GNU General Public License v2.0
6 stars 16 forks source link

Notification users incase of error/warning/info for http requests #182

Closed harunurhan closed 7 years ago

harunurhan commented 7 years ago

I think it's good to show user a little bit of information when we the record loading or tickets etc and also notify them if any request is failed.

When

v2

(after #192 is merged)

(Please add more or remove if you have something in mind @StellaCh @jmartinm )

UI

We could use https://github.com/scttcper/ngx-toastr for toast notifications

jmartinm commented 7 years ago

Also when getting the record and saving the record, GET and PUT requests it might happen that the user does not have permissions. We can show a message for that.

After extracting references it might also be good to show 'Extracting references...'

StellaCh commented 7 years ago

Yes, DESY also requested an 'Extracting References' message. Could be with a spinner.

A similar message 'Record is locked by user XXX' should be shown when someone tries to access the record but it is locked by someone else, see https://github.com/inspirehep/record-editor/issues/110

harunurhan commented 7 years ago

updated the list, @jmartinm @StellaCh

StellaCh commented 7 years ago

reverting: Reverting record to revision:

This description is not clear. What does it mean?