[24.0] Add GenericItem error handling

[ahmedhamidawan]

GenericItem was missing a onError function. Now, it catches backend errors and displays a Toast message for any failed operation. Fixes #18176

How to test the changes?

(Select all options that apply)

• I've included appropriate automated tests.
• This is a refactoring of components with existing test coverage.
• Instructions for manual testing are as follows:
  1. [add testing steps and prerequisites here if you didn't write automated tests covering all your changes]

License

• I agree to license these and all my past contributions to the core galaxy codebase under the MIT license.

[mvdbeek]

I think the toast is out of context here, could this be rendered in line ? As a user I couldn't really tell what the problem is in the screenshot. And I think all the delete/update methods in the component also need some error handling ?

[ahmedhamidawan]

I think the toast is out of context here, could this be rendered in line ? As a user I couldn't really tell what the problem is in the screenshot. And I think all the delete/update methods in the component also need some error handling ?

Oh I also added error handling for all the other update methods in GenericItem. I think adding a Title to the Toasts helps (which i did in the latest commit):

[mvdbeek]

Thank you @ahmedhamidawan!