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

Fixes broken storybook build #1317

Merged
merged 9 commits into from
May 12, 2023
Merged

Fixes broken storybook build #1317

merged 9 commits into from
May 12, 2023

Conversation

agiron123
Copy link
Contributor

@agiron123 agiron123 commented May 12, 2023 •
edited
Loading

Describe your changes and why you are making these changes

This PR accomplishes the following:
Upgrades storybook to version 7.0.11.
Fixes issues with storybook running.
Adds build storybook to ui build github action.

Related issue number (if any)

ENG-2970

Loom demo (if any)

https://www.loom.com/share/abb04fea262740e98c55b4eca5548cc3

Checklist before requesting a review

  • I have created a descriptive PR title. The PR title should complete the sentence "This PR...".
  • I have performed a self-review of my code.
  • I have included a small demo of the changes. For the UI, this would be a screenshot or a Loom video.
  • If this is a new feature, I have added unit tests and integration tests.
  • I have run the integration tests locally and they are passing.
  • I have run the linter script locally (See python3 scripts/run_linters.py -h for usage).
  • All features on the UI continue to work correctly.
  • Added one of the following CI labels:
    • run_integration_test: Runs integration tests
    • skip_integration_test: Skips integration tests (Should be used when changes are ONLY documentation/UI)

Upgrades storybook to version 7.0.11. Fixes issues with storybook run…
e5621f9 
…ning. Adds build storybook to ui build github action.
@agiron123 agiron123 added the skip_integration_test Skips required integration test (only documentation/UI changes) label May 12, 2023
agiron123
agiron123 commented May 12, 2023
View reviewed changes
src/ui/common/src/stories/DataListTable.stories.tsx
@@ -229,4 +230,8 @@ export const DataListTable: React.FC = () => {
);
};

export default DataListTable;
export default {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Have to add these export default blocks now or we get an error message.

agiron123
agiron123 commented May 12, 2023
View reviewed changes
src/ui/common/src/utils/SupportedIntegrations.tsx

const addingIntegrationLink = `${AqueductDocsLink}/integrations/adding-an-integration`;

export const SupportedIntegrations: ServiceInfoMap = {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Moved this out into it's own file since we're running into issues with the integrations.ts vs integrations.tsx.

@kenxu95 kenxu95 changed the title ENG-2970 Fixes broken storybook build May 12, 2023
kenxu95
kenxu95 approved these changes May 12, 2023
View reviewed changes
.github/workflows/ui-build.yml

- name: Build Storybook
working-directory: ./src/ui/common
run: npm run build-storybook
Copy link
Contributor

Choose a reason for hiding this comment

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

awesome!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

hopefully we can catch these issues when they come up in the future.

@agiron123 agiron123 merged commit a8abb16 into main May 12, 2023
likawind added a commit that referenced this pull request May 17, 2023
@likawind
Squashed commit of the following:
4196cf7 
commit e1fd9d7
Merge: 370665e 2fad420
Author: Wei Chen <[email protected]>
Date:   Wed May 17 14:19:11 2023 -0700

    Merge branch 'eng-2699-m1-refactor-wf-detail-pages-to-use-new' into eng-2699-m1-refactor-wf-detail-pages-to-use-new-2

commit 2fad420
Merge: b4485f4 676e81a
Author: Wei Chen <[email protected]>
Date:   Wed May 17 14:06:26 2023 -0700

    Merge branch 'eng-2699-m1-refactor-wf-detail-pages-to-use-new' of https://github.com/aqueducthq/aqueduct into eng-2699-m1-refactor-wf-detail-pages-to-use-new

commit b4485f4
Author: Wei Chen <[email protected]>
Date:   Mon Apr 24 15:43:51 2023 -0700

    rebase

commit d3a2669
Merge: e60ac09 18011cb
Author: Wei Chen <[email protected]>
Date:   Wed May 17 14:04:32 2023 -0700

    Merge branch 'eng-2712-m1-migrate-wf-edit-trigger-endpoints-to' into eng-2943-investigate-result-content-route-tests

commit 18011cb
Merge: ec515cf 8cb356c
Author: Wei Chen <[email protected]>
Date:   Wed May 17 14:02:47 2023 -0700

    Merge branch 'eng-2847-m1-implement-wfiddags-endpoint-with' into eng-2712-m1-migrate-wf-edit-trigger-endpoints-to

commit 8cb356c
Merge: ec94379 b3946a4
Author: Wei Chen <[email protected]>
Date:   Wed May 17 14:02:23 2023 -0700

    Merge branch 'main' of https://github.com/aqueducthq/aqueduct into eng-2847-m1-implement-wfiddags-endpoint-with

commit 676e81a
Author: Wei Chen <[email protected]>
Date:   Mon Apr 24 15:43:51 2023 -0700

    rebase

commit ec94379
Author: Wei Chen <[email protected]>
Date:   Wed May 17 13:28:38 2023 -0700

    comments

commit b3946a4
Author: kenxu95 <[email protected]>
Date:   Wed May 17 10:19:11 2023 -0700

    Move any overflow rows on the details header fields into an additional column (#1322)

commit e60ac09
Author: Wei Chen <[email protected]>
Date:   Tue May 16 20:04:49 2023 -0700

    fix

commit ec515cf
Merge: 1d0482a f81c065
Author: Wei Chen <[email protected]>
Date:   Tue May 16 17:37:04 2023 -0700

    Merge branch 'eng-2847-m1-implement-wfiddags-endpoint-with' into eng-2712-m1-migrate-wf-edit-trigger-endpoints-to

commit f81c065
Author: Wei Chen <[email protected]>
Date:   Tue May 16 17:34:45 2023 -0700

    lint

commit 1d0482a
Author: Wei Chen <[email protected]>
Date:   Tue May 16 17:29:44 2023 -0700

    works

commit 5ad5130
Author: Andre Giron <[email protected]>
Date:   Tue May 16 16:34:41 2023 -0700

    ENG-2979: Fixes bigquery and gcs dialogs that were crashing after file uploads. (#1326)

    Co-authored-by: Andre Giron <[email protected]>

commit 03aa76e
Author: Andre Giron <[email protected]>
Date:   Tue May 16 16:33:35 2023 -0700

    ENG-2979 S3 Dialog Validation Fixes (#1328)

    Co-authored-by: Andre Giron <[email protected]>

commit f63c5f3
Author: kenxu95 <[email protected]>
Date:   Tue May 16 16:24:14 2023 -0700

    Fix Snowflake save missing schema bug (#1331)

commit a458ce6
Author: Wei Chen <[email protected]>
Date:   Tue May 16 15:27:52 2023 -0700

    works

commit f529c5f
Author: Saurav Chhatrapati <[email protected]>
Date:   Tue May 16 13:36:12 2023 -0400

    Fixes cloudpickle serialization for Python 3.7 (#1324)

commit c335fd5
Author: Saurav Chhatrapati <[email protected]>
Date:   Tue May 16 12:15:48 2023 -0400

    Adds error checking for whether big query dataset exists (#1319)

commit d3521a4
Author: kenxu95 <[email protected]>
Date:   Mon May 15 20:58:16 2023 -0700

    Add ability to parameterize the save operator (#1320)

commit 942dec1
Author: kenxu95 <[email protected]>
Date:   Mon May 15 15:36:21 2023 -0700

    [UI] Merge the Conda resource into the Aqueduct Server  (#1311)

commit ccf4e17
Author: Wei Chen <[email protected]>
Date:   Mon May 15 14:57:19 2023 -0700

    [2/n] Adds gh actions to publish test pypi packages (#1262)

commit fb67044
Author: kenxu95 <[email protected]>
Date:   Mon May 15 12:04:48 2023 -0700

    Fix bug where duplicate fields showing in resource details dropdown (#1321)

commit cfeeb2f
Author: kenxu95 <[email protected]>
Date:   Mon May 15 11:45:31 2023 -0700

    Fix generated docstrings when viewing resource methods in Jupyter (#1315)

commit b544252
Author: eunice-chan <[email protected]>
Date:   Fri May 12 21:26:55 2023 -0500

    Eng 2884 test node routes (#1299)

    Co-authored-by: Eunice Chan <[email protected]>
    Co-authored-by: Wei Chen <[email protected]>

commit a8abb16
Author: Andre Giron <[email protected]>
Date:   Fri May 12 16:58:59 2023 -0700

    Fixes broken storybook build (#1317)

    Co-authored-by: Andre Giron <[email protected]>

commit d6ed5ca
Author: kenxu95 <[email protected]>
Date:   Fri May 12 10:56:42 2023 -0700

    Update the workflow "No Run" logo (#1312)

commit c9fec70
Author: Chenggang Wu <[email protected]>
Date:   Fri May 12 09:30:54 2023 -0700

    README typo fix (#1313)

commit b686014
Author: Andre Giron <[email protected]>
Date:   Thu May 11 12:49:53 2023 -0700

    ENG-2767 react hook form submission (#1232)

    Co-authored-by: Andre Giron <[email protected]>
    Co-authored-by: Chenggang Wu <[email protected]>
    Co-authored-by: kenxu95 <[email protected]>

commit 86269ec
Author: kenxu95 <[email protected]>
Date:   Thu May 11 12:09:42 2023 -0700

    Add appropriate workflow count to our connected notification cards and details page (#1302)

commit 7c75ad6
Author: kenxu95 <[email protected]>
Date:   Thu May 11 10:45:52 2023 -0700

    Update the artifact storage card presentation to be consistent with the other data integrations (#1297)

commit 4f9c227
Author: Andre Giron <[email protected]>
Date:   Wed May 10 16:55:23 2023 -0700

    ENG_2960: Fix broken documentation link for Aqueduct demo resource. (#1306)

    Co-authored-by: Andre Giron <[email protected]>

commit 6341917
Author: Andre Giron <[email protected]>
Date:   Wed May 10 16:28:23 2023 -0700

    Release version 0.3.2 (#1307)

    Co-authored-by: Vikram Sreekanti <[email protected]>
    Co-authored-by: Andre Giron <[email protected]>
@vsreekanti vsreekanti deleted the eng-2970-storybook-is-broken branch May 19, 2023 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kenxu95 kenxu95 kenxu95 approved these changes

Assignees
No one assigned
Labels
skip_integration_test Skips required integration test (only documentation/UI changes)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@agiron123 @kenxu95