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

Upgrade step to add ‘Show Toolbar’ permission. #127

Merged
merged 2 commits into from Jul 13, 2017

Conversation

Projects
None yet
3 participants
@agitator
Member

agitator commented Jul 7, 2017

No description provided.

Show outdated Hide outdated CHANGES.rst
@@ -10,7 +10,8 @@ Breaking changes:
New features:
- *add item here*
- Added ‘Show Toolbar’ permission.

This comment has been minimized.

@mauritsvanrees

mauritsvanrees Jul 10, 2017

Member

Here too, no 'pretty' quotes please. :-)

@mauritsvanrees

mauritsvanrees Jul 10, 2017

Member

Here too, no 'pretty' quotes please. :-)

@thet

This comment has been minimized.

Show comment
Hide comment
@thet

thet Jul 13, 2017

Member

LGTM

Member

thet commented Jul 13, 2017

LGTM

@thet thet merged commit 226558a into master Jul 13, 2017

3 of 4 checks passed

Plone Jenkins CI - pull-request-5.0 Please run the job, click here ----------->
Details
Changelog verifier Entry found
Details
Plone Contributors Agreement verifier All users have signed it
Details
Plone Jenkins CI - pull-request-5.1 Job finished with success status
Details

@thet thet deleted the show_toolbar_permission branch Jul 13, 2017

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