-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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 for issue #4092: Remove unused field from Project model #4431
Fix for issue #4092: Remove unused field from Project model #4431
Conversation
Thanks for looking at this, I think you made this PR with an old version of the repo (that's why the conflicts), do you mind opening a new PR with an updated branch?. Let me know if you need help to do so. |
@Alig1493 You should rebase on master and fix the merge conflicts. |
@stsewd @safwanrahman I'll look into it. |
@stsewd @safwanrahman I've looked into it to the best of my ability. Would you like to take a look into it? |
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.
LGTM, thanks!
model_name='project', | ||
name='comment_moderation', | ||
), | ||
migrations.AlterField( |
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.
Why this Altering?
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.
The rest of alters are from missing migrations (just changes in help text or options) #4201
Looks good. r+ |
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.
Thanks! This looks good to me!
Bumped the migration name, and will merge this 👍 |
Codecov Report
@@ Coverage Diff @@
## master #4431 +/- ##
==========================================
- Coverage 76.21% 76.21% -0.01%
==========================================
Files 158 158
Lines 10019 10017 -2
Branches 1265 1265
==========================================
- Hits 7636 7634 -2
Misses 2039 2039
Partials 344 344
|
Close #4092