feat: show error update notebook fail

[NguyenHoangSon96]

Closes #6999

Screen.Recording.2025-04-02.at.17.40.21.mov

Checklist

Authors and Reviewer(s), please verify the following:

• A PR description, regardless of the triviality of this change, that communicates the value of this PR
• Well-formatted conventional commit messages that provide context into the change
• Documentation updated or issue created (provide link to issue/PR)
• Signed CLA (if not already signed)
• Feature flagged, if applicable

[wdoconnell]

Remocal deployment failure. Re-running CI.

[wdoconnell]

@NguyenHoangSon96 Looks like CI flaked, please force push to re-run this.

[NguyenHoangSon96]

@NguyenHoangSon96 Looks like CI flaked, please force push to re-run this.

@wdoconnell
I have triggered CI. It's looking good now
Please check again

[NguyenHoangSon96]

After fixing the result is recorded in the description above ⬆️

[blegesse-w]

I might be missing context here, but from what I can tell, this PR adds error notifications, but I am not seeing anything that address the original issue - the UI still showing "Autosaved" when an error occurs. Should there be code that updates the UI state to remove the "Autosaved" indicator when errors happen?

[NguyenHoangSon96]

I might be missing context here, but from what I can tell, this PR adds error notifications, but I am not seeing anything that address the original issue - the UI still showing "Autosaved" when an error occurs. Should there be code that updates the UI state to remove the "Autosaved" indicator when errors happen?

Hi @blegesse-w
I'm not really sure, because I didn't see the "autosaved" label should be removed when an error happens in the "Expected behavior" and "Requested behaviour " section
As I understand the "autosaved" label is just there to tell the users that this page has an autosave feature