Update test: plone.app.discussion #3002
Conversation
Fix TTW Bundle compilation broken.
…pgraded if upgrade steps are available.
plone.staticresources auto upgrade
Apply types controlpanel fix to master.
…n reset (#2994) * Test if events fired after auto login have the correct principal * At this point the user is not logged in yet, so we get it from the membership_tool. * Update changelog * Remove adapter registration for IUserLoggedInEvent * Fix unregistering of adapter * Deal with cases where username/email were not the userid
|
@ksuess 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: With this simple comment all the jobs will be started automatically. Happy hacking! |
|
Obsolete. |
|
Reopened pull request: tests need to take plone.app.discussion workflow stati into account, even for replaced pull request in plone.app.discussion plone/plone.app.discussion#166 |
|
@ksuess 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: With this simple comment all the jobs will be started automatically. Happy hacking! |
plone.app.discussion introduces an additional workflow. This change takes the additional workflow states into account.
ksuess
force-pushed
the
plone.app.discussion-moderation
branch
from
cdfa540
to
7af3319
Compare
plone.app.discussion introduces an additional workflow. This change takes the additional workflow states into account.
See plone/plone.app.discussion#164