-
Notifications
You must be signed in to change notification settings - Fork 209
Refactor meetup/utils.py #655
Comments
@SanketDG Can I take this up? |
@PragatiVerma18 Assigned! |
@PragatiVerma18 Hi Any updates? |
Hello @SanketDG can I take up the issue! |
Sure! Assigned you @tejaswini22199 |
@sakshi1499 I removed multiple instances of payload and added only one of them. Can you help me in figuring out why it is failing tests while I'm merging. |
There was some issue with the last PR merge maybe that is why. Also, Your PR hasn't followed the commit guidelines so there are failing tests. @tejaswini22199 |
Hello @SanketDG @sakshi1499, |
Description
A lot of code in
meetup/utils.py
can be refactored, especially the jwt auth exchangeDefinition of Done
Estimation
3 hours
The text was updated successfully, but these errors were encountered: