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

Show speaker status on checkin result pages #324

Merged

Conversation

jandd
Copy link
Contributor

@jandd jandd commented Apr 10, 2024

  • add error code to validation error for checked in attendees
  • add speaker alert on checkin result page templates for manual and QR code checkin
  • add attendee.views.is_speaker function to check whether an attendee is a speaker for the current event
  • add the "is_speaker" context variable in CheckInAttendeeView and CheckInAttendeeUrlView
  • reformat changed Python files with isort and black

- add error code to validation error for checked in attendees
- add speaker alert on checkin result page templates for manual and QR
  code checkin
- add attendee.views.is_speaker function to check whether an attendee is
  a speaker for the current event
- add the "is_speaker" context variable in CheckInAttendeeView and
  CheckInAttendeeUrlView
- reformat changed Python files with isort and black
@jandd jandd merged commit 79e625a into development Apr 10, 2024
1 check passed
@jandd jandd deleted the feature/show-speaker-status-on-checkin-result-pages branch April 10, 2024 16:21
@coveralls
Copy link

Pull Request Test Coverage Report for Build 8634493043

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 4 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.1%) to 96.871%

Files with Coverage Reduction New Missed Lines %
attendee/views.py 4 98.78%
Totals Coverage Status
Change from base Build 8626951393: -0.1%
Covered Lines: 3787
Relevant Lines: 3899

💛 - Coveralls

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