[Bookings] Make network call to update booking note

[adborbas]

Closes WOOMOB-1617

Description

Adds calling the actual endpoint to update the note.

Test Steps

1. Download and install the app.
2. Login to a store with bookings.
3. Open the details of a booking.
4. Note what the current text for the note is. It might be just the placeholder.
5. Tap on the note.
6. Edit the note.
7. Tap done.
8. Wait a couple of seconds.
9. Force close the app.
10. Navigate to the same booking and validate that the note is the updated one.

Screenshots

---

• I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

[dangermattic]

	1 Message
📖	This PR contains changes to Tracks-related logic. Please ensure (author and reviewer) the following are completed: The tracks events must be validated in the Tracks system. Verify the internal Tracks spreadsheet has also been updated. Please consider registering any new events. The PR must be assigned the category: tracks label.

Generated by 🚫 Danger

[wpmobilebot]

📲 You can test the changes from this Pull Request in WooCommerce iOS Prototype by scanning the QR code below to install the corresponding build.

	App Name	WooCommerce iOS Prototype
	Build Number	pr16359-eb7d0f7
	Version	23.7
	Bundle ID	com.automattic.alpha.woocommerce
	Commit	eb7d0f7
	Installation URL	194urq553kiq8

Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

[itsmeichigo]

This works as described ✅

Regarding the UX, I'm concerned about the lack of progress indicator when saving the note. Also, the optimistic update of the note seems more complicated than necessary. How about showing a loading indicator in place of the Done button when it's tapped? Then we'll need to display the error toast in that same screen instead of on the detail screen.

[itsmeichigo]

💡 Making onCommit async and return an optional Error would let us show the loading indicator on this screen and display error when needed.

[adborbas]

Both addressed in b92bafd

[adborbas]

@itsmeichigo could you please take an other look?

[itsmeichigo]

This works nicely 💯 Thanks for the updates!

[itsmeichigo]

Merging now to include this change into my last PR.