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

Add behavior to hide modules from trainings. #669

Merged
merged 2 commits into from
Nov 17, 2023
Merged

Conversation

thet
Copy link
Member

@thet thet commented Nov 15, 2023

@thet thet requested a review from reinhardt November 16, 2023 08:47
@thet thet force-pushed the scrum-1573--hide-modules branch 2 times, most recently from 79ef584 to c9c2d73 Compare November 16, 2023 17:25
@thet
Copy link
Member Author

thet commented Nov 16, 2023

@reinhardt the test error is probably a test isolation error. locally the tests run through.
Should i add a unittest.skip to get this thing out of the door?

@reinhardt
Copy link
Contributor

Fixed!

@@ -78,6 +78,9 @@ def setUp(self):
fti_module.behaviors = tuple(
list(fti_module.behaviors) + ["euphorie.hide_from_training"]
)
from plone.dexterity.schema import SCHEMA_CACHE

SCHEMA_CACHE.invalidate("euphorie.module")
Copy link
Member Author

Choose a reason for hiding this comment

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

wow, thanks. didn't think of the schema cache!

Copy link
Contributor

Choose a reason for hiding this comment

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

I wasn't even aware it existed - I did a quick step through and got lucky because I noticed it. 😄

@thet thet merged commit 147eb13 into main Nov 17, 2023
1 check passed
@thet thet deleted the scrum-1573--hide-modules branch November 17, 2023 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants