Skip to content

Conversation

@nngu6036
Copy link

@nngu6036 nngu6036 commented May 3, 2019

No description provided.

@mister-roboto
Copy link

@nngu6036 thanks for creating this Pull Request and help improve Plone!

To ensure that these changes do not break other parts of Plone, the Plone test suite matrix needs to pass.

Whenever you feel that the pull request is ready to be tested, either start all jenkins jobs pull requests by yourself, or simply add a comment in this pull request stating:

@jenkins-plone-org please run jobs

With this simple comment all the jobs will be started automatically.

Happy hacking!

@djay
Copy link
Member

djay commented May 3, 2019

@nngu6036 There is a test for uninstall already. It would not be hard to add one more to test reinstall to this file. https://github.com/plone/plone.app.multilingual/blob/master/src/plone/app/multilingual/tests/test_uninstall.py. This would prevent the problem from happening again.

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