Skip to content

Conversation

@erral
Copy link
Member

@erral erral commented Jun 22, 2018

⚠️ #545 needs to be merged before this one. ⚠️


Like in #337 but with transition names, action titles and state titles in @worfklow endpoint

@erral erral force-pushed the translate-workflow-history branch 2 times, most recently from f642fdd to 80a90c4 Compare June 23, 2018 10:38
@erral
Copy link
Member Author

erral commented Jun 23, 2018

#545 needs to be merged before this one.

Copy link
Member

@lukasgraf lukasgraf left a comment

Choose a reason for hiding this comment

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

Just one minor details, otherwise looks good to me! 👍 🎉

self.portal_url = self.portal.absolute_url()
setRoles(self.portal, TEST_USER_ID, ['Manager'])

set_available_languages()
Copy link
Member

Choose a reason for hiding this comment

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

These three shouldn't be necessary any more since this is now done in the PloneRestApiDXLayer

@erral erral force-pushed the translate-workflow-history branch from 80a90c4 to 6031505 Compare June 23, 2018 11:13
@erral erral force-pushed the translate-workflow-history branch from 6031505 to d94e6a0 Compare June 23, 2018 14:06
@coveralls
Copy link

coveralls commented Jun 23, 2018

Coverage Status

Coverage decreased (-0.3%) to 96.188% when pulling 15ecf4f on translate-workflow-history into 2fd577b on master.

@erral erral changed the title [WIP] Translate workflow history titles Translate workflow history titles Jun 23, 2018
@tisto tisto self-requested a review June 24, 2018 07:34
@tisto tisto merged commit c92af77 into master Jun 24, 2018
@tisto tisto deleted the translate-workflow-history branch June 24, 2018 07:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants