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

Update security.rst, missing brackets in oauth bloc #1809

Closed
wants to merge 1 commit into from

Conversation

hakimoa
Copy link

@hakimoa hakimoa commented Feb 17, 2022

Add brakets to okta oauth configuration bloc

Description

ADDITIONAL INFORMATION

  • Has associated issue:
  • Is CRUD MVC related.
  • Is Auth, RBAC security related.
  • Changes the security db schema.
  • Introduces new feature
  • Removes existing feature

Add brakets to okta oauth configuration bloc
@codecov
Copy link

codecov bot commented Feb 17, 2022

Codecov Report

Merging #1809 (a2802d8) into master (8a18281) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1809   +/-   ##
=======================================
  Coverage   77.11%   77.11%           
=======================================
  Files          56       56           
  Lines        8229     8229           
=======================================
  Hits         6346     6346           
  Misses       1883     1883           
Flag Coverage Δ
python 77.11% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8a18281...a2802d8. Read the comment docs.

@davidnateberg
Copy link
Contributor

davidnateberg commented Feb 21, 2022

Hello, I actually already created a PR for this
#1798

Funny enough you did the same thing I did the first time with the brackets

@dpgaspar
Copy link
Owner

This was probably fixed by: #1798

Closing, feel free to reopen if you think it's not fixed

@dpgaspar dpgaspar closed this Apr 12, 2022
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.

None yet

3 participants