Skip to content

Sorting support for JWA#6742

Merged
google-oss-prow[bot] merged 2 commits into
kubeflow:masterfrom
arrikto:feature-elena-wa-support-sorting
Nov 15, 2022
Merged

Sorting support for JWA#6742
google-oss-prow[bot] merged 2 commits into
kubeflow:masterfrom
arrikto:feature-elena-wa-support-sorting

Conversation

@elenzio9

Copy link
Copy Markdown
Contributor

This PR:

  • Introduce the sorting functionality in the common code.
  • Enable sorting in JWA.

A screenshot of how it will look like:
image

The plan is to make VWA and TWA support sorting as well.

Related issue: #6460

@elenzio9
elenzio9 force-pushed the feature-elena-wa-support-sorting branch from 06eee57 to 0f49500 Compare November 15, 2022 09:50
* Add sorting functionality in the common code.

Signed-off-by: Elena Zioga <elena@arrikto.com>
* Enable sorting functionality in JWA.

Signed-off-by: Elena Zioga <elena@arrikto.com>
@kimwnasptd

Copy link
Copy Markdown
Member

Built the image and it works as expected.

We can go with this approach for now, of doing the sorting in the frontend, but we'll really need to start looking at having some informer/caching mechanism in the backend.

/lgtm
/approve

@google-oss-prow

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: kimwnasptd

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@google-oss-prow
google-oss-prow Bot merged commit 0ddc032 into kubeflow:master Nov 15, 2022
@kimwnasptd
kimwnasptd deleted the feature-elena-wa-support-sorting branch November 15, 2022 14:54
maroroman pushed a commit to maroroman/kubeflow that referenced this pull request Feb 7, 2023
* web-apps(front): Add sorting in common code (kubeflow#6460)

* Add sorting functionality in the common code.

Signed-off-by: Elena Zioga <elena@arrikto.com>

* jwa(front): Enable sorting in JWA (kubeflow#6460)

* Enable sorting functionality in JWA.

Signed-off-by: Elena Zioga <elena@arrikto.com>

Signed-off-by: Elena Zioga <elena@arrikto.com>
Adembc pushed a commit to Adembc/notebooks that referenced this pull request Jun 22, 2024
* web-apps(front): Add sorting in common code (#6460)

* Add sorting functionality in the common code.

Signed-off-by: Elena Zioga <elena@arrikto.com>

* jwa(front): Enable sorting in JWA (#6460)

* Enable sorting functionality in JWA.

Signed-off-by: Elena Zioga <elena@arrikto.com>

Signed-off-by: Elena Zioga <elena@arrikto.com>
Adembc pushed a commit to Adembc/notebooks that referenced this pull request Jun 22, 2024
* web-apps(front): Add sorting in common code (#6460)

* Add sorting functionality in the common code.

Signed-off-by: Elena Zioga <elena@arrikto.com>

* jwa(front): Enable sorting in JWA (#6460)

* Enable sorting functionality in JWA.

Signed-off-by: Elena Zioga <elena@arrikto.com>

Signed-off-by: Elena Zioga <elena@arrikto.com>
Adembc pushed a commit to Adembc/notebooks that referenced this pull request Jun 22, 2024
* web-apps(front): Add sorting in common code (#6460)

* Add sorting functionality in the common code.

Signed-off-by: Elena Zioga <elena@arrikto.com>

* jwa(front): Enable sorting in JWA (#6460)

* Enable sorting functionality in JWA.

Signed-off-by: Elena Zioga <elena@arrikto.com>

Signed-off-by: Elena Zioga <elena@arrikto.com>
Adembc pushed a commit to Adembc/notebooks that referenced this pull request Jun 23, 2024
* web-apps(front): Add sorting in common code (#6460)

* Add sorting functionality in the common code.

Signed-off-by: Elena Zioga <elena@arrikto.com>

* jwa(front): Enable sorting in JWA (#6460)

* Enable sorting functionality in JWA.

Signed-off-by: Elena Zioga <elena@arrikto.com>

Signed-off-by: Elena Zioga <elena@arrikto.com>
Adembc pushed a commit to Adembc/notebooks that referenced this pull request Jun 23, 2024
* web-apps(front): Add sorting in common code (#6460)

* Add sorting functionality in the common code.

Signed-off-by: Elena Zioga <elena@arrikto.com>

* jwa(front): Enable sorting in JWA (#6460)

* Enable sorting functionality in JWA.

Signed-off-by: Elena Zioga <elena@arrikto.com>

Signed-off-by: Elena Zioga <elena@arrikto.com>
andyatmiami pushed a commit to andyatmiami/kubeflow-notebooks that referenced this pull request Aug 13, 2025
* web-apps(front): Add sorting in common code (kubeflow/kubeflow#6460)

* Add sorting functionality in the common code.

Signed-off-by: Elena Zioga <elena@arrikto.com>

* jwa(front): Enable sorting in JWA (kubeflow/kubeflow#6460)

* Enable sorting functionality in JWA.

Signed-off-by: Elena Zioga <elena@arrikto.com>

Signed-off-by: Elena Zioga <elena@arrikto.com>
thesuperzapper pushed a commit to kubeflow/notebooks that referenced this pull request Aug 18, 2025
* web-apps(front): Add sorting in common code (kubeflow/kubeflow#6460)

* Add sorting functionality in the common code.

Signed-off-by: Elena Zioga <elena@arrikto.com>

* jwa(front): Enable sorting in JWA (kubeflow/kubeflow#6460)

* Enable sorting functionality in JWA.

Signed-off-by: Elena Zioga <elena@arrikto.com>

Signed-off-by: Elena Zioga <elena@arrikto.com>
utruong309 pushed a commit to utruong309/notebooks that referenced this pull request Jan 8, 2026
* web-apps(front): Add sorting in common code (kubeflow/kubeflow#6460)

* Add sorting functionality in the common code.

Signed-off-by: Elena Zioga <elena@arrikto.com>

* jwa(front): Enable sorting in JWA (kubeflow/kubeflow#6460)

* Enable sorting functionality in JWA.

Signed-off-by: Elena Zioga <elena@arrikto.com>

Signed-off-by: Elena Zioga <elena@arrikto.com>
utruong309 pushed a commit to utruong309/notebooks that referenced this pull request Jan 8, 2026
* web-apps(front): Add sorting in common code (kubeflow/kubeflow#6460)

* Add sorting functionality in the common code.

Signed-off-by: Elena Zioga <elena@arrikto.com>

* jwa(front): Enable sorting in JWA (kubeflow/kubeflow#6460)

* Enable sorting functionality in JWA.

Signed-off-by: Elena Zioga <elena@arrikto.com>

Signed-off-by: Elena Zioga <elena@arrikto.com>
Signed-off-by: utruong309 <uyenthutruong09@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants