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 a way to track organizer issues #939

Merged
merged 7 commits into from
Feb 1, 2024
Merged

Conversation

amakarudze
Copy link
Contributor

Every now and then community members report issues to do with Django Girls workshop organizers through email. However, since email is separate from the CMS, these issues fall through the cracks and are only uncovered when the same organizer has another issue reported against them.

This pull request introduces:

  • An OrganizerIssue model to track issues to do with organizers.
  • Adds a field is_blacklisted to the user model.
  • Adds status of organizer being blacklisted to the string representation of user to make it easy for Awesomeness Ambassador to know when an organizer has been blacklisted.
  • Also add checks when new event application is received to see if any of the listed organizers are blacklisted.
  • Adds a field to event application admin to easily show this to Awesomeness Ambassador.
  • Tests for these changes [TO DO].

@amakarudze amakarudze linked an issue Jan 31, 2024 that may be closed by this pull request
Copy link

codecov bot commented Jan 31, 2024

Codecov Report

Attention: 8 lines in your changes are missing coverage. Please review.

Comparison is base (6a05d70) 81.01% compared to head (e57bc27) 81.24%.
Report is 18 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #939      +/-   ##
==========================================
+ Coverage   81.01%   81.24%   +0.23%     
==========================================
  Files          95       98       +3     
  Lines        2860     2938      +78     
  Branches      269      271       +2     
==========================================
+ Hits         2317     2387      +70     
- Misses        492      499       +7     
- Partials       51       52       +1     
Files Coverage Δ
core/admin/__init__.py 100.00% <100.00%> (ø)
core/admin/forms/organizerissue.py 100.00% <100.00%> (ø)
core/admin/organizerissue.py 100.00% <100.00%> (ø)
core/admin/user.py 63.63% <ø> (ø)
core/models/organizerissue.py 100.00% <100.00%> (ø)
core/models/user.py 100.00% <100.00%> (ø)
organize/admin.py 60.00% <ø> (ø)
organize/models.py 91.42% <88.88%> (-0.24%) ⬇️
organize/views.py 63.15% <12.50%> (-8.28%) ⬇️

... and 4 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6a05d70...e57bc27. Read the comment docs.

core/models/organizerissue.py Outdated Show resolved Hide resolved
core/migrations/0006_auto_20240130_1644.py Outdated Show resolved Hide resolved
core/models/organizerissue.py Outdated Show resolved Hide resolved
core/models/organizerissue.py Outdated Show resolved Hide resolved
amakarudze and others added 4 commits February 1, 2024 15:46
Co-authored-by: Mark Walker <theshow@gmail.com>
Co-authored-by: Mark Walker <theshow@gmail.com>
Co-authored-by: Mark Walker <theshow@gmail.com>
Co-authored-by: Mark Walker <theshow@gmail.com>
Copy link
Collaborator

@marksweb marksweb left a comment

Choose a reason for hiding this comment

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

Think this looks good with those improvements in 👍

@amakarudze amakarudze merged commit b49dbec into DjangoGirls:main Feb 1, 2024
6 checks passed
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.

Add a way to track issues raised against organizers
2 participants