Skip to content

Conversation

@Varun-Sethu
Copy link
Member

PR adds the first stage of the required changes to the front-end in order to support operational transform. This PR captures SlateJS operations and propagates them to a central place for delivery to the server. The PR after this will focus on translating SlateJS operations into the CMS operation format + sending them to the server and the final PR will focus on applying operations sent from the server to the editor.

@Varun-Sethu Varun-Sethu requested a review from fafnirZ as a code owner October 31, 2022 12:25
@Varun-Sethu Varun-Sethu requested a review from lauraw0 October 31, 2022 12:26
@Varun-Sethu Varun-Sethu assigned jamest0 and unassigned jamest0 Oct 31, 2022
@Varun-Sethu Varun-Sethu requested a review from jamest0 October 31, 2022 12:27
@Varun-Sethu
Copy link
Member Author

Tests are failing because there appears to be something wrong with PIP implying the DB startup script couldn't be run and consequentially the backend testing env couldn't be created

@Varun-Sethu Varun-Sethu requested a review from zijizhu October 31, 2022 22:44
Copy link
Contributor

@zijizhu zijizhu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

Copy link
Contributor

@lauraw0 lauraw0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm! :)

@Varun-Sethu Varun-Sethu requested a review from a team as a code owner November 1, 2022 06:41
@Varun-Sethu Varun-Sethu force-pushed the varun/otFrontend/stage1 branch from 37b6f36 to ba6ee0f Compare November 1, 2022 06:43
@Varun-Sethu Varun-Sethu force-pushed the varun/otFrontend/stage1 branch from ba6ee0f to 88a69cc Compare November 1, 2022 06:50
@Varun-Sethu Varun-Sethu requested a review from lhjt November 2, 2022 09:27
@Varun-Sethu Varun-Sethu force-pushed the varun/otFrontend/stage1 branch from 4688b16 to f336dde Compare November 2, 2022 09:30
@Varun-Sethu Varun-Sethu force-pushed the varun/otFrontend/stage1 branch from f336dde to 5e9dcfc Compare November 2, 2022 09:33
@Varun-Sethu Varun-Sethu merged commit 90470b7 into main Nov 3, 2022
@Varun-Sethu Varun-Sethu deleted the varun/otFrontend/stage1 branch November 3, 2022 23:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants