Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

EZEE-3484: Create translated draft before edit if autosave is enabled #1714

Merged
merged 9 commits into from
Mar 9, 2021

Conversation

Nattfarinn
Copy link
Member

Question Answer
Tickets https://issues.ibexa.co/browse/EZEE-3484
Bug fix? yes
New feature? no
BC breaks? maybe
Tests pass? yes
Doc needed? no
License GPL-2.0

Checklist:

  • Coding standards ($ composer fix-cs)
  • Ready for Code Review

@Nattfarinn Nattfarinn changed the base branch from master to 2.2 February 26, 2021 10:49
Nattfarinn and others added 3 commits February 26, 2021 11:54
Co-authored-by: Paweł Niedzielski <pawel.niedzielski@ibexa.co>
Co-authored-by: Paweł Niedzielski <pawel.niedzielski@ibexa.co>
Copy link
Contributor

@Steveb-p Steveb-p left a comment

Choose a reason for hiding this comment

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

LGTM. I've got a minor, tiny remark, but it's not really that important.

src/lib/EventListener/ContentProxyCreateDraftListener.php Outdated Show resolved Hide resolved
Co-authored-by: Paweł Niedzielski <pawel.niedzielski@ibexa.co>
@adamwojs
Copy link
Member

adamwojs commented Mar 2, 2021

@Nattfarinn Code style needs to be fixed here

src/lib/EventListener/ContentProxyCreateDraftListener.php Outdated Show resolved Hide resolved
])
);
}

$event->setResponse($response);
}

public function translate(ContentProxyTranslateEvent $event): void
Copy link
Member

@adamwojs adamwojs Mar 4, 2021

Choose a reason for hiding this comment

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

We need test coverage for this code. See #1714 (comment)

Copy link
Member Author

Choose a reason for hiding this comment

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

True, true.

Copy link
Contributor

@tomaszszopinski tomaszszopinski left a comment

Choose a reason for hiding this comment

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

QA approved on Ibexa DXP 3.2.4-dev with patch.

@lserwatka lserwatka merged commit 4ee19bb into 2.2 Mar 9, 2021
@lserwatka lserwatka deleted the ezee-3484-translate-autosave branch March 9, 2021 08:05
@lserwatka
Copy link
Member

Could you merge it up?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
6 participants