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

Merge current master into devmain #606

Merged
merged 92 commits into from
Sep 30, 2022
Merged

Conversation

olga-union
Copy link
Contributor

TL;DR

Merge current master into devmain.

Type

  • Bug Fix
  • Feature
  • Plugin
  • Chore

Are all requirements met?

  • Code completed
  • Smoke tested
  • Unit tests added
  • Code documentation added
  • Any pending items have an associated Issue

Follow-up issue

NA

anrusina and others added 30 commits May 17, 2022 18:05
Signed-off-by: Nastya Rusina <nastya@union.ai>
Signed-off-by: Nastya Rusina <nastya@union.ai>
Signed-off-by: Nastya Rusina <nastya@union.ai>
Signed-off-by: Nastya Rusina <nastya@union.ai>
Signed-off-by: Nastya Rusina <nastya@union.ai>
Signed-off-by: Nastya Rusina <nastya@union.ai>
Signed-off-by: Nastya Rusina <nastya@union.ai>
Signed-off-by: Carina Ursu <carina@union.ai>

Co-authored-by: Carina Ursu <carina@union.ai>
Signed-off-by: eugenejahn <eugenejahnjahn@gmail.com>
* ci: move storybook related packages to top level
* ci: ensure that lint can be run in PR checks
* ci: allow to run tests linux setup

Signed-off-by: Nastya Rusina <nastya@union.ai>
Signed-off-by: eugenejahn <eugenejahnjahn@gmail.com>
#436)

* ci: ensure unified tsconfig and remove mocks,test,stories from build
* ci: move certificates into certificate folder inside script

Signed-off-by: Nastya Rusina <nastya@union.ai>
* fix: issue 398
* fix: prevent parent onclick event trigger

Signed-off-by: eugenejahn <eugenejahnjahn@gmail.com>
* fix: unable to view all the workflow versions

Signed-off-by: Carina Ursu <carina@union.ai>

* chore: cleanup

Signed-off-by: Carina Ursu <carina@union.ai>

Co-authored-by: Carina Ursu <carina@union.ai>
* chore(tsc): exclude test/stories/mock files only from build

Signed-off-by: Nastya Rusina <nastya@union.ai>
* chore: support for StructuredDataSet Input/Output type

Signed-off-by: Carina Ursu <carina@union.ai>

* chore: add tests

Signed-off-by: Carina Ursu <carina@union.ai>

* chore: add package

Signed-off-by: Carina Ursu <carina@union.ai>

* chore: resolving local package issue

Signed-off-by: Carina Ursu <carina@union.ai>

* chore: yarn lock

Signed-off-by: Carina Ursu <carina@union.ai>

* chore: stories

Signed-off-by: Carina Ursu <carina@union.ai>

* chore: fix associative arrays in copy functionality

Signed-off-by: Carina Ursu <carina@union.ai>

* chore: fix scalar stories

Signed-off-by: Carina Ursu <carina@union.ai>

* chore: oops

Signed-off-by: Carina Ursu <carina@union.ai>

* chore: cleanup

Signed-off-by: Carina Ursu <carina@union.ai>

* chore: revert old viewer to original

Signed-off-by: Carina Ursu <carina@union.ai>

* chore: fix

Signed-off-by: Carina Ursu <carina@union.ai>

* chore: remove non-null assertion

Signed-off-by: Carina Ursu <carina@union.ai>

* chore: fix types

Signed-off-by: Carina Ursu <carina@union.ai>

* chore: cleanup

Signed-off-by: Carina Ursu <carina@union.ai>

* chore: fix tests

Signed-off-by: Carina Ursu <carina@union.ai>

* chore: fix test types

Signed-off-by: Carina Ursu <carina@union.ai>

* chore: cleanup

Signed-off-by: Carina Ursu <carina@union.ai>

* chore: cleanup

Signed-off-by: Carina Ursu <carina@union.ai>

* chore: cleanup

Signed-off-by: Carina Ursu <carina@union.ai>

* chore: cleanup

Signed-off-by: Carina Ursu <carina@union.ai>

Co-authored-by: Carina Ursu <carina@union.ai>
…scode setup properly (#458)

* chore: separate AppInfo
* chore: vscode properly understands @flyteconsole
* chore: add basics/locale entry
* test: add storybooks and tests
* test: fix istanbul test-coverage issue by binding babel versions

Signed-off-by: Nastya Rusina <nastya@union.ai>
* ci: minimize babel presence

Signed-off-by: Nastya Rusina <nastya@union.ai>
* fix: update node executions to display map tasks
* fix: update map task logs styles
* test: add/update unit tests
* fix: fix flickering and unnecessary re-renders

Signed-off-by: Olga Nad <olga@union.ai>
* ci: resolve docker build issues
* chore: remove unneeded changes

Signed-off-by: Nastya Rusina <nastya@union.ai>
* chore: fix

Signed-off-by: Carina Ursu <carina@union.ai>

* chore: remove console

Signed-off-by: Carina Ursu <carina@union.ai>

* chore: add cherio o=to package

Signed-off-by: Carina Ursu <carina@union.ai>

* chore: add cherio o=to package

Signed-off-by: Carina Ursu <carina@union.ai>

* chore: fix start prod

Signed-off-by: Carina Ursu <carina@union.ai>

* chore: fix start prod

Signed-off-by: Carina Ursu <carina@union.ai>

* chore: cleanup

Signed-off-by: Carina Ursu <carina@union.ai>

* chore: cleanup

Signed-off-by: Carina Ursu <carina@union.ai>

Co-authored-by: Carina Ursu <carina@union.ai>
Signed-off-by: Nastya <55718143+anrusina@users.noreply.github.com>
Signed-off-by: Nastya <55718143+anrusina@users.noreply.github.com>
made the entities component become more generic that can support Workflow, Task, and others
added the inputs and outputs to task details page
added the versions table in task details page
added the version details page
added task details link in node side panel

Signed-off-by: eugenejahn <eugenejahnjahn@gmail.com>

Co-authored-by: Eugene Jahn <eugenejahnjahn@gmail.com>
Signed-off-by: Nastya Rusina <nastya@union.ai>

Co-authored-by: Nastya Rusina <nastya@union.ai>
* hotfix ci pipeline

Signed-off-by: Yuvraj <evalsocket@users.noreply.github.com>

* more changes

Signed-off-by: Yuvraj <evalsocket@users.noreply.github.com>

Co-authored-by: Yuvraj <evalsocket@users.noreply.github.com>
yindia and others added 26 commits July 25, 2022 19:07
Bumps [terser](https://github.com/terser/terser) from 4.8.0 to 4.8.1.
- [Release notes](https://github.com/terser/terser/releases)
- [Changelog](https://github.com/terser/terser/blob/master/CHANGELOG.md)
- [Commits](https://github.com/terser/terser/commits)

---
updated-dependencies:
- dependency-name: terser
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: Flyte-Bot <admin@flyte.org>

Signed-off-by: Flyte-Bot <admin@flyte.org>
Co-authored-by: flyte-bot <flyte-bot@users.noreply.github.com>
chore: fix searchbar X button

Signed-off-by: Carina Ursu <carina@union.ai>

Signed-off-by: Carina Ursu <carina@union.ai>
#562)

* fix: update timeline view to show dynamic wf internals on first render

Signed-off-by: Olga Nad <olga@union.ai>

* fix: update tests and clean up code

Signed-off-by: Olga Nad <olga@union.ai>

* fix: test

Signed-off-by: Olga Nad <olga@union.ai>

Signed-off-by: Olga Nad <olga@union.ai>
Signed-off-by: Carina Ursu <carina@union.ai>

Signed-off-by: Carina Ursu <carina@union.ai>
* fix: console showing subworkflows as unknown

Signed-off-by: Olga Nad <olga@union.ai>

* fix: replace comparison with lodash isEqual

Signed-off-by: Olga Nad <olga@union.ai>

* fix: remove checkIfObjectsAreSame

Signed-off-by: Olga Nad <olga@union.ai>

Signed-off-by: Olga Nad <olga@union.ai>
fix: dict lose trailing issue

Signed-off-by: James <james@union.ai>

Signed-off-by: James <james@union.ai>
* fix: disable launch button and show validation messaages

Signed-off-by: James <james@union.ai>

* fix: code clean

Signed-off-by: James <james@union.ai>

Signed-off-by: James <james@union.ai>
…572)

* fix: integrate timeline and graph tabs wrappers under one component

Signed-off-by: Olga Nad <olga@union.ai>

* fix: details tab in graph view and clean-up

Signed-off-by: Olga Nad <olga@union.ai>

* fix: old unit tests

Signed-off-by: Olga Nad <olga@union.ai>

* fix: temp filename change

Signed-off-by: Olga Nad <olga@union.ai>

* fix: final filename change

Signed-off-by: Olga Nad <olga@union.ai>

* fix: tsconfig

Signed-off-by: Olga Nad <olga@union.ai>

Signed-off-by: Olga Nad <olga@union.ai>
Bumps [moment](https://github.com/moment/moment) from 2.29.3 to 2.29.4.
- [Release notes](https://github.com/moment/moment/releases)
- [Changelog](https://github.com/moment/moment/blob/develop/CHANGELOG.md)
- [Commits](moment/moment@2.29.3...2.29.4)

---
updated-dependencies:
- dependency-name: moment
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* added none type in union type

Signed-off-by: eugenejahn <eugenejahnjahn@gmail.com>

* udpated the test case

Signed-off-by: eugenejahn <eugenejahnjahn@gmail.com>

* fix: fixed eslint

Signed-off-by: eugenejahn <eugenejahnjahn@gmail.com>

Signed-off-by: eugenejahn <eugenejahnjahn@gmail.com>
#578)

fix: correctly coerce defaultValue when rendering LaunchPlan details

Signed-off-by: Rahul Mehta <rahul@theoremlp.com>

Signed-off-by: Rahul Mehta <rahul@theoremlp.com>
fix: inputHelpers InputProps

Signed-off-by: Carina Ursu <carina@union.ai>

Signed-off-by: Carina Ursu <carina@union.ai>
* fix: fix test of launchform

Signed-off-by: James <james@union.ai>

* fix: inputHelpers tests regression

Signed-off-by: Olga Nad <olga@union.ai>

* fix: update tests description

Signed-off-by: Olga Nad <olga@union.ai>

* fix: spelling typo

Signed-off-by: Olga Nad <olga@union.ai>

Signed-off-by: James <james@union.ai>
Signed-off-by: Olga Nad <olga@union.ai>
Co-authored-by: Olga Nad <olga@union.ai>
Signed-off-by: Jason Porter <jason@union.ai>

Signed-off-by: Jason Porter <jason@union.ai>
Minor change

Signed-off-by: Jason Porter <jason@union.ai>

Signed-off-by: Jason Porter <jason@union.ai>
…#582)

Signed-off-by: Olga Nad <olga@union.ai>

Signed-off-by: Olga Nad <olga@union.ai>
Signed-off-by: Flyte-Bot <admin@flyte.org>

Signed-off-by: Flyte-Bot <admin@flyte.org>
Co-authored-by: flyte-bot <flyte-bot@users.noreply.github.com>
* fix: add BASE_URL to dev startup, open deeply nested urls

Signed-off-by: Carina Ursu <carina@union.ai>

* fix: remove debug code

Signed-off-by: Carina Ursu <carina@union.ai>

* chore: empty BASE_URL handling

Signed-off-by: Carina Ursu <carina@union.ai>

Signed-off-by: Carina Ursu <carina@union.ai>
* fix: add default disabled state for only mine filter

Signed-off-by: Olga Nad <olga@union.ai>

* fix: tests

Signed-off-by: Olga Nad <olga@union.ai>

* fix: use api context to default filter state

Signed-off-by: Olga Nad <olga@union.ai>

* fix: revert test updates

Signed-off-by: Olga Nad <olga@union.ai>

Signed-off-by: Olga Nad <olga@union.ai>
Signed-off-by: Flyte-Bot <admin@flyte.org>

Signed-off-by: Flyte-Bot <admin@flyte.org>
Co-authored-by: flyte-bot <flyte-bot@users.noreply.github.com>
* progress

* Fixed

* Removed debug code

Signed-off-by: Jason Porter <jason@union.ai>

Signed-off-by: Jason Porter <jason@union.ai>
chore: enable deeplinks in development

Signed-off-by: Carina Ursu <carina@union.ai>

Signed-off-by: Carina Ursu <carina@union.ai>
Signed-off-by: Jason Porter <jason@union.ai>

Signed-off-by: Jason Porter <jason@union.ai>
@codecov
Copy link

codecov bot commented Sep 30, 2022

Codecov Report

Merging #606 (2139738) into devmain (d38b98b) will decrease coverage by 2.26%.
The diff coverage is 45.62%.

@@             Coverage Diff             @@
##           devmain     #606      +/-   ##
===========================================
- Coverage    71.05%   68.79%   -2.27%     
===========================================
  Files          426      442      +16     
  Lines         9849    10478     +629     
  Branches      1673     1796     +123     
===========================================
+ Hits          6998     7208     +210     
- Misses        2851     3270     +419     
Impacted Files Coverage Δ
...le/src/components/Entities/EntityDetailsHeader.tsx 86.11% <ø> (ø)
...nsole/src/components/Entities/EntityExecutions.tsx 96.77% <ø> (-0.29%) ⬇️
...s/VersionDetails/EntityVersionDetailsContainer.tsx 0.00% <0.00%> (ø)
.../zapp/console/src/components/Entities/constants.ts 100.00% <ø> (ø)
...es/zapp/console/src/components/Entities/strings.ts 100.00% <ø> (ø)
...cutionDetails/NodeExecutionDetailsPanelContent.tsx 91.48% <ø> (ø)
...cutions/ExecutionDetails/RelaunchExecutionForm.tsx 100.00% <ø> (ø)
...ls/TaskExecutionNodeRenderer/TaskExecutionNode.tsx 0.00% <0.00%> (ø)
...Executions/ExecutionDetails/Timeline/taskNames.tsx 0.00% <ø> (-60.00%) ⬇️
...components/Executions/ExecutionDetails/strings.tsx 100.00% <ø> (ø)
... and 104 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@jsonporter jsonporter merged commit f8cb277 into devmain Sep 30, 2022
@jsonporter jsonporter deleted the olga-union/update-devmain branch September 30, 2022 22:24
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.

10 participants