-
Notifications
You must be signed in to change notification settings - Fork 343
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
Migrate Elyra extensions to support JupyterLab 4.x #3201
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm assuming the build failures are a temporary hurdle. Thanks for dealing with the @lresende!
af4a4de
to
ab95e71
Compare
Signed-off-by: Luciano Resende <lresende@apple.com>
Signed-off-by: Luciano Resende <lresende@apple.com>
Signed-off-by: Luciano Resende <lresende@apple.com>
a5b1686
to
6410a77
Compare
I have made some progress here, but I would appreciate if I could get some help on the UI pieces, in particular on UI-components... @marthacryan @karlaspuldaro @ajbozarth and others feel free to send prs to this branch as time permits... I completely understand if you are not able to. |
110767d
to
7567988
Compare
Signed-off-by: Luciano Resende <lresende@apple.com>
Signed-off-by: Luciano Resende <lresende@apple.com>
7567988
to
95332e5
Compare
.github/workflows/build.yml
Outdated
@@ -41,7 +41,12 @@ jobs: | |||
/home/runner/.cache/Cypress | |||
key: ${{ runner.os }}-yarn-${{ hashFiles('**/yarn.lock') }} | |||
- name: Install | |||
run: yarn install --frozen-lockfile && tsc -v | |||
run: | | |||
corepack prepare yarn@3.5.0 --activate |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This line is causing build failure, maybe it's best to use a stable version of yarn
hello! is this pull request still being worked on? it would be amazing to have support for JL4 now that JL3 has an official sunset date. Is there any support required from the community? Thank you! |
There is a pr against this pr, let me merge that over the weekend and then others can jump in to help finish any remaining items |
thanks for the quick response! |
hi @lresende, thank you so much for this work! Where are these 'remaining items' located? I would love to figure out ways to help! (at least testing it) |
@ederign they are opened as PR to the branch, here: lresende#1 |
Updated LauncherModel to accommodate changes in @lumino/algorithm. Resolved build errors related to missing properties and import issues.
Resolved type errors related to missing children property in Dropzone.tsx, ExpandableComponent.tsx, and ExpandableErrorDialog.tsx. Fixed import issues in FormEditor.tsx by addressing missing members and correcting import statements. Improved TypeScript type inference and resolved implicit 'any' types in parameters and binding elements in JSONComponent.tsx.
Updated CodeSnippetDisplay class to extend MetadataDisplay with the correct props type Replace usage of deprecated ContentFactory, but still exist issue when missing documentation.
tried to fix the first code cell creation error, but still have issue when inserting cell to notebook widget
Just figured out how to properly add codecell.
Fixed some method calling base on documentation.
Resolved final error in pipeline editor. Now it's able to build.
- .value.text does not work for updating the content of Cell, Replaced with .setSource. - Refresh no longer exists, replaced with redo. Can't find any documentation about this change. - fixed creation of preview editor
This reverts commit ac10f57.
This reverts commit b2630c7.
This reverts commit 9ab11eb.
trying to adjust workflow build issue
--------- Signed-off-by: Harshad Reddy Nalla <hnalla@redhat.com> Signed-off-by: shalberd <21118431+shalberd@users.noreply.github.com> Co-authored-by: Guilherme Caponetto <638737+caponetto@users.noreply.github.com> Co-authored-by: Eder Ignatowicz <ignatowicz@gmail.com> Co-authored-by: Edson Tirelli <ed.tirelli@gmail.com> Co-authored-by: Harshad Reddy Nalla <hnalla@redhat.com> Co-authored-by: Sven Thoms <21118431+shalberd@users.noreply.github.com>
Hi @lresende, this migration still being worked on? should we expect changes in the near future? |
@Ekzz0 we need to resolve some of these conflicts, but, at a high level, the branch is working with JupyterLab 4.2.5 |
What changes were proposed in this pull request?
Migrate Elyra extensions to support JupyterLab 4.x
Fixes #3137
How was this pull request tested?
Building and running on JupyterLab 4.x
Developer's Certificate of Origin 1.1