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

QT: add progress plan Graph #171

Merged
merged 5 commits into from
Nov 10, 2023
Merged

QT: add progress plan Graph #171

merged 5 commits into from
Nov 10, 2023

Conversation

shishovKA
Copy link
Contributor

@shishovKA shishovKA commented Oct 24, 2023

https://nda.ya.ru/t/WnmZs6Kl6vUrHD

how to check this huge PR:

1st commit is the biggest one, but it contains only "copy-paste" files for yql-like graph

other commits are more informative, they are contain:

  • installing libs
  • hiding JS webworker functionality (for now decided not to use worker cause build lib troubles)
  • reworking graph nodes onClick handlers (open YT table or YT operation)
  • replacing some internal component with ours analogs

@shishovKA shishovKA force-pushed the qt-plan branch 4 times, most recently from ec1db61 to 3fb4e5d Compare November 1, 2023 09:53
@shishovKA shishovKA force-pushed the qt-plan branch 2 times, most recently from 18d6c50 to 587c289 Compare November 7, 2023 08:22
@shishovKA shishovKA changed the title Qt plan [Draft! do not review] QT: add progress plan Graph Nov 7, 2023
@shishovKA shishovKA marked this pull request as ready for review November 7, 2023 09:01
@shishovKA shishovKA merged commit e45218a into main Nov 10, 2023
7 checks passed
@shishovKA shishovKA deleted the qt-plan branch November 10, 2023 14:15
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.

2 participants