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 devmain #607

Merged
merged 99 commits into from
Oct 3, 2022
Merged

fix devmain #607

merged 99 commits into from
Oct 3, 2022

Conversation

olga-union
Copy link
Contributor

TL;DR

Fixing the build on devmain.

Type

  • Bug Fix
  • Feature
  • Plugin

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 <[email protected]>
Signed-off-by: Carina Ursu <[email protected]>

Co-authored-by: Carina Ursu <[email protected]>
* 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 <[email protected]>
#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 <[email protected]>
* fix: issue 398
* fix: prevent parent onclick event trigger

Signed-off-by: eugenejahn <[email protected]>
* fix: unable to view all the workflow versions

Signed-off-by: Carina Ursu <[email protected]>

* chore: cleanup

Signed-off-by: Carina Ursu <[email protected]>

Co-authored-by: Carina Ursu <[email protected]>
* chore(tsc): exclude test/stories/mock files only from build

Signed-off-by: Nastya Rusina <[email protected]>
* chore: support for StructuredDataSet Input/Output type

Signed-off-by: Carina Ursu <[email protected]>

* chore: add tests

Signed-off-by: Carina Ursu <[email protected]>

* chore: add package

Signed-off-by: Carina Ursu <[email protected]>

* chore: resolving local package issue

Signed-off-by: Carina Ursu <[email protected]>

* chore: yarn lock

Signed-off-by: Carina Ursu <[email protected]>

* chore: stories

Signed-off-by: Carina Ursu <[email protected]>

* chore: fix associative arrays in copy functionality

Signed-off-by: Carina Ursu <[email protected]>

* chore: fix scalar stories

Signed-off-by: Carina Ursu <[email protected]>

* chore: oops

Signed-off-by: Carina Ursu <[email protected]>

* chore: cleanup

Signed-off-by: Carina Ursu <[email protected]>

* chore: revert old viewer to original

Signed-off-by: Carina Ursu <[email protected]>

* chore: fix

Signed-off-by: Carina Ursu <[email protected]>

* chore: remove non-null assertion

Signed-off-by: Carina Ursu <[email protected]>

* chore: fix types

Signed-off-by: Carina Ursu <[email protected]>

* chore: cleanup

Signed-off-by: Carina Ursu <[email protected]>

* chore: fix tests

Signed-off-by: Carina Ursu <[email protected]>

* chore: fix test types

Signed-off-by: Carina Ursu <[email protected]>

* chore: cleanup

Signed-off-by: Carina Ursu <[email protected]>

* chore: cleanup

Signed-off-by: Carina Ursu <[email protected]>

* chore: cleanup

Signed-off-by: Carina Ursu <[email protected]>

* chore: cleanup

Signed-off-by: Carina Ursu <[email protected]>

Co-authored-by: Carina Ursu <[email protected]>
…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 <[email protected]>
* ci: minimize babel presence

Signed-off-by: Nastya Rusina <[email protected]>
* 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 <[email protected]>
* ci: resolve docker build issues
* chore: remove unneeded changes

Signed-off-by: Nastya Rusina <[email protected]>
* chore: fix

Signed-off-by: Carina Ursu <[email protected]>

* chore: remove console

Signed-off-by: Carina Ursu <[email protected]>

* chore: add cherio o=to package

Signed-off-by: Carina Ursu <[email protected]>

* chore: add cherio o=to package

Signed-off-by: Carina Ursu <[email protected]>

* chore: fix start prod

Signed-off-by: Carina Ursu <[email protected]>

* chore: fix start prod

Signed-off-by: Carina Ursu <[email protected]>

* chore: cleanup

Signed-off-by: Carina Ursu <[email protected]>

* chore: cleanup

Signed-off-by: Carina Ursu <[email protected]>

Co-authored-by: Carina Ursu <[email protected]>
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 <[email protected]>

Co-authored-by: Eugene Jahn <[email protected]>
* hotfix ci pipeline

Signed-off-by: Yuvraj <[email protected]>

* more changes

Signed-off-by: Yuvraj <[email protected]>

Co-authored-by: Yuvraj <[email protected]>
james-union and others added 25 commits August 22, 2022 09:39
fix: dict lose trailing issue

Signed-off-by: James <[email protected]>

Signed-off-by: James <[email protected]>
* fix: disable launch button and show validation messaages

Signed-off-by: James <[email protected]>

* fix: code clean

Signed-off-by: James <[email protected]>

Signed-off-by: James <[email protected]>
…572)

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

Signed-off-by: Olga Nad <[email protected]>

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

Signed-off-by: Olga Nad <[email protected]>

* fix: old unit tests

Signed-off-by: Olga Nad <[email protected]>

* fix: temp filename change

Signed-off-by: Olga Nad <[email protected]>

* fix: final filename change

Signed-off-by: Olga Nad <[email protected]>

* fix: tsconfig

Signed-off-by: Olga Nad <[email protected]>

Signed-off-by: Olga Nad <[email protected]>
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] <[email protected]>

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

Signed-off-by: eugenejahn <[email protected]>

* udpated the test case

Signed-off-by: eugenejahn <[email protected]>

* fix: fixed eslint

Signed-off-by: eugenejahn <[email protected]>

Signed-off-by: eugenejahn <[email protected]>
#578)

fix: correctly coerce defaultValue when rendering LaunchPlan details

Signed-off-by: Rahul Mehta <[email protected]>

Signed-off-by: Rahul Mehta <[email protected]>
fix: inputHelpers InputProps

Signed-off-by: Carina Ursu <[email protected]>

Signed-off-by: Carina Ursu <[email protected]>
* fix: fix test of launchform

Signed-off-by: James <[email protected]>

* fix: inputHelpers tests regression

Signed-off-by: Olga Nad <[email protected]>

* fix: update tests description

Signed-off-by: Olga Nad <[email protected]>

* fix: spelling typo

Signed-off-by: Olga Nad <[email protected]>

Signed-off-by: James <[email protected]>
Signed-off-by: Olga Nad <[email protected]>
Co-authored-by: Olga Nad <[email protected]>
Signed-off-by: Jason Porter <[email protected]>

Signed-off-by: Jason Porter <[email protected]>
Minor change

Signed-off-by: Jason Porter <[email protected]>

Signed-off-by: Jason Porter <[email protected]>
Signed-off-by: Flyte-Bot <[email protected]>

Signed-off-by: Flyte-Bot <[email protected]>
Co-authored-by: flyte-bot <[email protected]>
* fix: add BASE_URL to dev startup, open deeply nested urls

Signed-off-by: Carina Ursu <[email protected]>

* fix: remove debug code

Signed-off-by: Carina Ursu <[email protected]>

* chore: empty BASE_URL handling

Signed-off-by: Carina Ursu <[email protected]>

Signed-off-by: Carina Ursu <[email protected]>
* fix: add default disabled state for only mine filter

Signed-off-by: Olga Nad <[email protected]>

* fix: tests

Signed-off-by: Olga Nad <[email protected]>

* fix: use api context to default filter state

Signed-off-by: Olga Nad <[email protected]>

* fix: revert test updates

Signed-off-by: Olga Nad <[email protected]>

Signed-off-by: Olga Nad <[email protected]>
Signed-off-by: Flyte-Bot <[email protected]>

Signed-off-by: Flyte-Bot <[email protected]>
Co-authored-by: flyte-bot <[email protected]>
* progress

* Fixed

* Removed debug code

Signed-off-by: Jason Porter <[email protected]>

Signed-off-by: Jason Porter <[email protected]>
chore: enable deeplinks in development

Signed-off-by: Carina Ursu <[email protected]>

Signed-off-by: Carina Ursu <[email protected]>
Signed-off-by: Jason Porter <[email protected]>

Signed-off-by: Jason Porter <[email protected]>
Signed-off-by: Olga Nad <[email protected]>
@codecov
Copy link

codecov bot commented Oct 1, 2022

Codecov Report

Merging #607 (da2fd51) into devmain (f8cb277) will increase coverage by 0.29%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           devmain     #607      +/-   ##
===========================================
+ Coverage    68.79%   69.08%   +0.29%     
===========================================
  Files          442      440       -2     
  Lines        10478    10434      -44     
  Branches      1796     1789       -7     
===========================================
  Hits          7208     7208              
+ Misses        3270     3226      -44     

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 7e80dba into devmain Oct 3, 2022
@jsonporter jsonporter deleted the olga-union/fix-devmain branch October 3, 2022 16:29
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