Make BookingNotesView bindable

[adborbas]

Fixes an issue where the booking note did not update on selection change on iPad.

Description

The root of the issue was that we were holding onto the notes as a @State and so iPadOS did not pick up the change. I've made that a Binding and have also adopted to the patrn of Content used in the BookingDetailViewModel.

Test Steps

1. Use an iPad
2. Login to a store with CIAB.
3. Navigate to Bookings.
4. Open a booking with a note or add to one.
5. Open multiple bookings in the split, validate each show's own note.

Screenshots

Before	After

---

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

[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	pr16398-bfc28be
	Version	23.7
	Bundle ID	com.automattic.alpha.woocommerce
	Commit	bfc28be
	Installation URL	6cah0t29u8oog

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

[RafaelKayumov]

Tested. Works as described 👍

Nit: I wonder if it's necessary to add the NotesContent + BookingNotesRowView wrappers since the MultilineEditableTextRow.value is turned into binding?

Would it be enough just to keep string note and make it @Published and then pass it to MultilineEditableTextRow as binding?

final class BookingDetailsViewModel: ObservableObject {
    ...
    @Published var note = ""
    ...
}

func updateDisplayProperties(from booking: Booking) {
...
    noteText = booking.note
...
}

func bookingNotesView() -> some View {
        MultilineEditableTextRow(value: $viewModel.noteText,
                                 placeholder: Localization.bookingNotesRowText,
                                 detailTitle: Localization.bookingNoteNavbarText) { newNote in
            return await viewModel.updateNote(to: newNote)
        }
    }

[adborbas]

@RafaelKayumov, yes, that is a totally valid option as well. But I've opted for the NoteContent to better fit into how the other sections are working. They contain the data as well and do not access it directly from the viewModel.

[dangermattic]

	1 Warning
⚠️	This PR is assigned to the milestone 23.8. This milestone is due in less than 2 days. Please make sure to get it merged by then or assign it to a milestone with a later deadline.

Generated by 🚫 Danger