Skip to content
This repository has been archived by the owner on Jan 24, 2024. It is now read-only.

[docs] Add docs for OAuth credentials #1964

Conversation

Demogorgon314
Copy link
Member

Motivation

#1728 Introduced tenant for OAuth, and #1926 introduced group ID for OAuth, we should add docs.

Documentation

Check the box below.

Need to update docs?

  • doc-required

    (If you need help on updating docs, create a doc issue)

  • no-need-doc

    (Please explain why)

  • doc

    (If this PR contains doc changes)

@Demogorgon314 Demogorgon314 added the doc This pr contains a document label Jul 17, 2023
@Demogorgon314 Demogorgon314 self-assigned this Jul 17, 2023
@codecov
Copy link

codecov bot commented Jul 17, 2023

Codecov Report

Merging #1964 (5427099) into master (1dda879) will increase coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1964      +/-   ##
============================================
+ Coverage     17.72%   17.74%   +0.02%     
- Complexity      751      752       +1     
============================================
  Files           195      195              
  Lines         14155    14155              
  Branches       1322     1322              
============================================
+ Hits           2509     2512       +3     
+ Misses        11463    11461       -2     
+ Partials        183      182       -1     

see 1 file with indirect coverage changes

@Demogorgon314 Demogorgon314 marked this pull request as ready for review July 26, 2023 14:15
@BewareMyPower BewareMyPower merged commit 308d75a into streamnative:master Jul 31, 2023
21 checks passed
Demogorgon314 added a commit to Demogorgon314/kop that referenced this pull request Aug 14, 2023
### Motivation
streamnative#1728 Introduced tenant for
OAuth, and streamnative#1926 introduced
group ID for OAuth, we should add docs.

(cherry picked from commit 308d75a)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants