jaredhanson / passport-facebook

Facebook authentication strategy for Passport and Node.js.
https://www.passportjs.org/packages/passport-facebook/?utm_source=github&utm_medium=referral&utm_campaign=passport-facebook&utm_content=about
MIT License
1.29k stars 446 forks source link

Adding locale in options constructor Strategy #266

Closed rafaelharus closed 1 year ago

rafaelharus commented 4 years ago

READ THIS FIRST!

Are you implementing a new feature?

A locale can be set in the options that will be used to build the url. A defined locale makes user-liked page categories have a default language.

Requests for new features should first be discussed on the developer forum. This allows the community to gather feedback and assess whether or not there is an existing way to achieve the desired functionality.

If it is determined that a new feature needs to be implemented, include a link to the relevant discussion along with the pull request.

Is this a security patch?

Do not open pull requests that might have security implications. Potential security vulnerabilities should be reported privately to jaredhanson@gmail.com. Once any vulerabilities have been repaired, the details will be disclosed publicly in a responsible manner. This also allows time for coordinating with affected parties in order to mitigate negative consequences.

If neither of the above two scenarios apply to your situation, you should open a pull request. Delete this paragraph and the text above, and fill in the information requested below.

Checklist

coveralls commented 4 years ago

Coverage Status

Coverage remained the same at 100.0% when pulling 691dd47566b23a010ba2541ee8a2f72cb79d32cc on rafaelharus:master into 9337b1b9788c11160387bbc831b43fc8f024c8de on jaredhanson:master.

coveralls commented 4 years ago

Coverage Status

Coverage remained the same at 100.0% when pulling 691dd47566b23a010ba2541ee8a2f72cb79d32cc on rafaelharus:master into 9337b1b9788c11160387bbc831b43fc8f024c8de on jaredhanson:master.

mariojrrc commented 4 years ago

@jaredhanson could you have a look at this PR?