Skip to content

feat(authentication,router,admin)!: captcha/cors settings#233

Merged
kkopanidis merged 1 commit intomainfrom
cors-captcha
Apr 7, 2023
Merged

feat(authentication,router,admin)!: captcha/cors settings#233
kkopanidis merged 1 commit intomainfrom
cors-captcha

Conversation

@kkopanidis
Copy link
Copy Markdown
Contributor

refactor(authentication)!: remove captcha provider settings
feat(router): add captcha providers settings
feat(router,admin): add cors settings

What kind of change does this PR introduce? (check at least one)

  • Bugfix
  • Feature
  • Code style update
  • Refactor
  • Build-related changes
  • Other, please describe:

Does this PR introduce a breaking change? (check one)

  • Yes
  • No

If yes, please describe the impact and migration path for existing applications:

The PR fulfills these requirements:

  • It's submitted to the master branch
  • When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx, where "xxx" is the issue number)

If adding a new feature, the PR's description includes:

  • A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:

feat(router): add captcha providers settings
feat(router,admin): add cors settings
@kkopanidis kkopanidis requested review from kon14 and ssellas April 7, 2023 09:09
@kkopanidis kkopanidis merged commit 75cd5aa into main Apr 7, 2023
@kkopanidis kkopanidis deleted the cors-captcha branch April 7, 2023 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants