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

EZP-29694: Changed publishing to publish last modified translation #1005

Merged
merged 1 commit into from
May 30, 2019

Conversation

ViniTou
Copy link
Contributor

@ViniTou ViniTou commented May 29, 2019

Question Answer
Tickets https://jira.ez.no/browse/EZP-29694 / https://jira.ez.no/browse/EZEE-2661
Bug fix? no
New feature? no
BC breaks? yes
Tests pass? yes
Doc needed? yes
License GPL-2.0

This change how default behavior for publish works on our UI side. Instead of publishing all translations contained in draft (which right now cannot be edited outside of PAPI/REST) we are publishing draft only in initialLanguageCode of draft.

Other PRs:

ezsystems/repository-forms#296

Uses:

ezsystems/ezpublish-kernel#2615

Checklist:

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

Copy link
Contributor

@SylvainGuittard SylvainGuittard left a comment

Choose a reason for hiding this comment

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

Thanks @ViniTou

@micszo micszo self-assigned this May 30, 2019
Copy link
Member

@micszo micszo left a comment

Choose a reason for hiding this comment

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

Tested together with ezsystems/ezpublish-kernel#2615.

@micszo micszo removed their assignment May 30, 2019
@alongosz alongosz changed the title EZP-29694: Publish Version only in initialLanguageCode of draft EZP-29694: Changed publishing to publish last modified translation May 30, 2019
@alongosz alongosz merged commit 45e40aa into 1.5 May 30, 2019
@alongosz alongosz deleted the EZEE-2661-publish-modified-langauge-only branch May 30, 2019 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

6 participants