-
-
Notifications
You must be signed in to change notification settings - Fork 14
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
2.2 #80
2.2 #80
Conversation
Development
Development
Changed format for user created & joined
…uirement in change for perms requirement
minor fixes
Added censor and fixed requirements
and reformatted also changed ai a tiny bit.
@JasonLovesDoggo can you re-review please |
doesn't matter we are still waiting on db migration and changes. |
…els DB reformatted.
and updated reqs
This reverts commit e264d67.
trivia/tags: used enumerate vs +1 lewis: added failsafe. feat: added tag lb
style: fixed to pep8 reformat.
Database migrated |
fixed all requested changes
|
||
before_roles.reverse() | ||
before_roles.pop() | ||
|
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.
messssssssssssssssssssssyyyyyyyyy
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.
not really
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 whole section
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.
that's true, but as I said I didn't code it and also don't have time to refactor code
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.
otherwise I would just do a few simple list comprehensions and clean it up, but that all needs time to code and test afterwards
for before_name in before.permissions: | ||
before_.append(before_name) | ||
|
||
for after_name in after.permissions: |
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.
huhhhh all seems very messy and unsimplified
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.
I didnt write that code, but yeah can be done in a list comprehension
needs db migrating don't merge yet