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

feat: add generator submodule #155

Closed
wants to merge 25 commits into from
Closed

feat: add generator submodule #155

wants to merge 25 commits into from

Conversation

shuowu
Copy link
Contributor

@shuowu shuowu commented Jul 27, 2021

In this PR:

  • adds code generator (@okta/generator) to generate samples and test harness app (only package.json for now)
  • adds generated samples (under generated folder)
    • @okta/samples.react.okta-hosted-login (github sample)
    • @okta/samples.react.custom-login (github sample)
    • @okta/samples.react.doc-embedded-widget (doc sample)
    • @okta/samples.react.doc-direct-auth (doc sample)
  • adds samples e2e (@okta/test.e2e-samples), it currently runs against protractor (mostly brought from the okta-oidc-tck repo)
  • binds sample e2e with bacon

@shuowu shuowu marked this pull request as draft July 27, 2021 21:40
@shuowu-okta shuowu-okta marked this pull request as ready for review July 30, 2021 18:06
@@ -0,0 +1,63 @@
/*!
* Copyright (c) 2015-2018, Okta, Inc. and/or its affiliates. All rights reserved.
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: can extend banners script's files allowlist

Copy link
Contributor

@denysoblohin-okta denysoblohin-okta left a comment

Choose a reason for hiding this comment

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

LGTM

To run this application, you first need to clone this repo:

```bash
git clone https://github.com/okta/samples-js-react.git
Copy link
Contributor

Choose a reason for hiding this comment

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

Should new generated samples in this repo replace samples-js-react repo?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It is the first step for samples auto-generation, in this PR, it grabs samples from the samples-js-react repo and developer docs, so in the future, when there is a change in the SDK repo, the PR should also update the generated samples at the same time, then downstream repos can just pull the latest changes without manual changes.

generator/_templates/samples/config/package.json.t Outdated Show resolved Hide resolved
shuowu and others added 2 commits August 27, 2021 14:38
Co-authored-by: denysoblohin-okta <72614880+denysoblohin-okta@users.noreply.github.com>
eng-prod-CI-bot-okta pushed a commit that referenced this pull request Aug 27, 2021
OKTA-417465
<<<Jenkins Check-In of Tested SHA: 195793c for eng_productivity_ci_bot_okta@okta.com>>>
Artifact: okta-react
Files changed count: 164
PR Link: "#155"
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