[16.0][IMP] fieldservice_calendar: Sync description #1199
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Factor out recurse_order_calendar context key checks * Use record sets more * Tests: Factor out order creation & use tests.Form instead of plain create
Propagate description updates between FSM order & calendar event. 1 is an HTML field, the other is not; we handle conversion. Before this change, description was only propagated at calendar event creation, and without text➔html conversion.
|
Hi @hparfr, |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR contains 2 commits: 1 to modernize code related to this field sync; 1 which adds description sync.
Propagate description updates between FSM order & calendar event.
1 is an HTML field, the other is not; we handle conversion.
Before this change, description was only propagated at calendar event creation, and without text➔html conversion.
To test:
FSM order with multiline description & schedule:
Calendar event with that multiline description (thanks to text➔html conversion):
