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.

Sign up for GitHub

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

Jump to bottom

Jans config api fixes #1940

Merged
merged 18 commits into from
Jul 29, 2022
Merged

Jans config api fixes #1940

merged 18 commits into from
Jul 29, 2022

Conversation

pujavs
Copy link
Contributor

@pujavs pujavs commented Jul 29, 2022 •
edited by mo-auto
Loading

Prepare

Description

Changes in scim-plugin to add new SCIM config attributes.

Target issue

closes #1875

Implementation Details

Modified plugin DTO and swagger to add new config attributes

Test and Document the changes

  • Static code analysis has been run locally and issues have been fixed
  • Relevant unit and integration tests have been added/updated
  • Relevant documentation has been updated if any (i.e. user guides, installation and configuration guides, technical design docs etc)

Closes #1941,
Closes #1948,

pujavs added 18 commits July 22, 2022 15:54
@pujavs
bug(jans-config-api): fixed swagger format issue
9d444bd
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
4c253a2 
…ns-config-api-fixes
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
f6eb80b 
…ns-config-api-fixes
@pujavs
fix(jans-config-api): fixed due to couchbase clustter change
ac97cc6
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
cd554ae 
…ns-config-api-fixes
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
f4bd7aa 
…ns-config-api-fixes
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
b313aae 
…ns-config-api-fixes
@pujavs
feat(jans-config-api): new endpoint to get UmaResource based on assoc…
c8003fd 
…iatedClient
@pujavs
fix(jans-config-api): swagger spec fix for client attributes
fc5ebe0
@pujavs
fix(jans-config-api): reverted the local test properties
77d19bb
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
d4f8194 
…ns-config-api-fixes
@pujavs
test(jans-config-api): commented test case
a977d1f
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
15dee7f 
…ns-config-api-fixes
@pujavs
feat(jans-config-api): scim config endpoint enhancment
b409633
@pujavs
feat(jans-config-api): swagger and DTO change for new fields for scim…
1498d74 
… config endpoint
@pujavs
Merge branch 'jans-config-api-fixes' of https://github.com/JanssenPro…
6940645 
…ject/jans into jans-config-api-fixes
@pujavs
feat(jans-config-api): swagger and DTO change for new fields for scim…
3ac7ea1 
… config endpoint
@pujavs
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
8db181b 
…ns-config-api-fixes
@pujavs pujavs requested a review from yuriyz as a code owner July 29, 2022 13:14
@mo-auto mo-auto mentioned this pull request Jul 29, 2022
Closed
@mo-auto
Copy link
Member

mo-auto commented Jul 29, 2022

Error: Hi @pujavs, You did not reference an open issue in your PR. I attempted to create an issue for you.
Please update that issues' title and body and make sure I correctly referenced it in the above PRs body.

@mo-auto mo-auto added the comp-jans-config-api Component affected by issue or PR label Jul 29, 2022
@pujavs pujavs mentioned this pull request Jul 29, 2022
Closed
@yuriyz yuriyz merged commit 757b22f into main Jul 29, 2022
@yuriyz yuriyz deleted the jans-config-api-fixes branch July 29, 2022 13:33
@mo-auto mo-auto mentioned this pull request Jul 29, 2022
Closed
@mo-auto
Copy link
Member

mo-auto commented Jul 29, 2022

Error: Hi @pujavs, You did not reference an open issue in your PR. I attempted to create an issue for you.
Please update that issues' title and body and make sure I correctly referenced it in the above PRs body.

@mo-auto mo-auto mentioned this pull request Aug 30, 2022
Merged
@mo-auto3 mo-auto3 mentioned this pull request Aug 30, 2022
Merged
@mo-auto mo-auto mentioned this pull request Nov 1, 2022
Merged
@mo-auto3 mo-auto3 mentioned this pull request Nov 1, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yuriyz yuriyz yuriyz approved these changes

Assignees
No one assigned
Labels
comp-jans-config-api Component affected by issue or PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

fix: Jans config api fixes -autocreated fix: Jans config api fixes -autocreated
3 participants
@pujavs @mo-auto @yuriyz