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

fix(idx): allow flow entry point request when use generic remediator (beta) #1251

Closed
wants to merge 2 commits into from

Conversation

shuowu
Copy link
Contributor

@shuowu shuowu commented Jul 5, 2022

No description provided.

@shuowu shuowu changed the base branch from master to 6.7 July 5, 2022 14:59
@@ -51,7 +92,8 @@ describe('remediators/GenericRemediator', () => {
});
});

describe('nested inputs', () => {
// Re-enable when add client side validation in GenericRemediator
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add a JIRA for this and put the ID in the comment?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This feature may not be needed depend on which approach siw-next will pick to implement client side validation (auth-js vs ajv). Will the jira still be needed for uncertain works?

@shuowu-okta shuowu-okta force-pushed the sw-fix-generic-remediator-flow-entry-request branch from 29ba8d8 to be411d0 Compare July 5, 2022 22:40
eng-prod-CI-bot-okta pushed a commit that referenced this pull request Jul 5, 2022
…(beta)

OKTA-512298
<<<Jenkins Check-In of Tested SHA: be411d0 for eng_productivity_ci_bot_okta@okta.com>>>
Artifact: okta-auth-js
Files changed count: 4
PR Link: #1251
eng-prod-CI-bot-okta pushed a commit that referenced this pull request Jul 5, 2022
OKTA-512298
<<<Jenkins Check-In of Tested SHA: be411d0 for eng_productivity_ci_bot_okta@okta.com>>>
Artifact: okta-auth-js
Files changed count: 4
PR Link: #1251
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.

4 participants