Skip to content
This repository was archived by the owner on May 27, 2021. It is now read-only.

apis for add and remove roles #53

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

debamitra
Copy link
Collaborator

What kind of change does this PR introduce

What is the current behavior

What is the new behavior

Checklist for this PR

  • I have read contributing guidelines.
  • I executed npm run lint locally to format my code.
  • I added revelant Tests to verify my code.
  • All tests npm run test ran successfully.
  • Pull request is towards develop branch.
  • Added myself to contributors table
  • Documentation
  • Ready to be merged

@debamitra debamitra changed the title User role ap is apis for add and remove roles Oct 9, 2019
@@ -1,6 +1,8 @@
const jwt = require('jsonwebtoken');

const User = require('../models').User;
const Role = require('../models').Role;
const UserRole = require('../models').UserRole;

Copy link
Contributor

Choose a reason for hiding this comment

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

Isn't Babel configured for this project?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

What's Babel and what do we need it for?

Copy link
Member

Choose a reason for hiding this comment

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

Babel is a JavaScript transpiler, it converts modern JavaScript into the older versions that most browsers understand

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants