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
fix: Return only accepted session and public speaker call by default #6328
fix: Return only accepted session and public speaker call by default #6328
Conversation
@@ -86,20 +86,29 @@ def before_get_object(self, view_kwargs): | |||
""" | |||
if view_kwargs.get('event_identifier'): | |||
try: | |||
event = self.session.query(Event).filter_by(identifier=view_kwargs['event_identifier']).one() | |||
event = self.session.query(Event).filter_by(identifier=view_kwargs['event_identifier'], | |||
privacy='public').one() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
public and accepted
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@iamareebjamal Speaker calls are an invitation to submit a proposal for being a speaker at the event. They are either private
or public
. I didn't get the context of accepted here. Did you mean making changes for speakers to only return accepted speakers?
Codecov Report
@@ Coverage Diff @@
## development #6328 +/- ##
===============================================
- Coverage 65.24% 65.22% -0.03%
===============================================
Files 287 287
Lines 14731 14736 +5
===============================================
Hits 9611 9611
- Misses 5120 5125 +5
Continue to review full report at Codecov.
|
)" This reverts commit 3987cd3.
Fixes #5998
Checklist
development
branch.