express-jwt icon indicating copy to clipboard operation
express-jwt copied to clipboard

Fix readme inconsistency

Open vitalii-bulyzhyn opened this issue 1 year ago • 0 comments

By submitting a PR to this repository, you agree to the terms within the Auth0 Code of Conduct. Please see the contributing guidelines for how to create and submit a high-quality PR for this repo.

Description

I noted little inconsistency in the readme docs. It's said that for migration to V6 instead of default export:

var jwt = require("express-jwt");

It should be:

var { expressjwt: jwt } = require("express-jwt");

So, I fixed that in 2 places in Readme

Testing

Describe how this can be tested by reviewers. Be specific about anything not tested and reasons why. If this library has unit and/or integration testing, tests should be added for new functionality and existing tests should complete without errors.

Please include any manual steps for testing end-to-end or functionality not covered by unit/integration tests.

Also include details of the environment this PR was developed in (language/platform/browser version).

  • [x] This change adds test coverage for new/changed/fixed functionality

Checklist

  • [x] I have added documentation for new/changed functionality in this PR or in auth0.com/docs
  • [x] All active GitHub checks for tests, formatting, and security are passing
  • [x] The correct base branch is being used, if not the default branch

vitalii-bulyzhyn avatar Feb 28 '24 22:02 vitalii-bulyzhyn