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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test for manual_evaluation exercise messages behavior #1580

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

luchotc
Copy link
Contributor

@luchotc luchotc commented Feb 26, 2021

Creating a draft PR for current issue with messages.

This is due to exercises default_scope 馃槙.

@luchotc
Copy link
Contributor Author

luchotc commented Feb 26, 2021

@flbulgarelli I'm not sure which is the best way to tackle this. We could limit messages like .where(..exercises: organization.exercises) and that would do the trick, but it feels like a nasty patch 馃槙

@flbulgarelli
Copy link
Member

Hi! I think I need a little more context here 馃槄

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.

None yet

2 participants