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

All conventions are followed in resources files #937

Closed
wants to merge 2 commits into from

Conversation

gitpurva
Copy link

@gitpurva gitpurva commented Dec 19, 2020

Description

Mentioned changes in the resources file to follow same convention.

Fixes #898

  • My PR follows the style guidelines of this project
  • I have performed a self-review of my own code or materials

@devkapilbansal
Copy link
Member

Continue of #905
@PrashanthPuneriya please review it too

class ListAdmins(Resource):
@classmethod
@jwt_required

Copy link
Member

Choose a reason for hiding this comment

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

this empty line isn't needed right?

@devkapilbansal
Copy link
Member

@gitpurva there are some merge conflicts, please resolve them

@NenadPantelic
Copy link
Contributor

HI, @gitpurva do you want to finish this PR or it should be closed?

@devkapilbansal devkapilbansal added the Status: Changes Requested Changes are required to be done by the PR author. label Feb 18, 2021
@devkapilbansal
Copy link
Member

Closing due to inactivity. @gitpurva feel free to ping again if you want to continue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Changes Requested Changes are required to be done by the PR author.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make mentioned changes in resources files to follow the same convention
4 participants