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

[manager-components]: Asynchronous datagrid columns #14942

Open
1 task done
qpavy opened this issue Jan 15, 2025 · 2 comments
Open
1 task done

[manager-components]: Asynchronous datagrid columns #14942

qpavy opened this issue Jan 15, 2025 · 2 comments

Comments

@qpavy
Copy link
Contributor

qpavy commented Jan 15, 2025

Have you already checked if a similar item is present on manager-components?

  • Yes, I have already checked the existing components/hooks/utils.

What do you expect from this request?

Component

Description

Is it possible to improve datagrid by adding a new feature to allow loading of datas which is not on datagrid iceberg call ?
The goal would be to use column filter which will be created here #14244 and add a property for each column to allow the use of additionnal api endpoint for the column.
When the datagrid load with the column hidden, no need to call the other api endpoint, only the main iceberg call. But once the column is displayed, we call asynchronously the api endpoint set in the property for each line, a squeleton appear waiting for the result and once the result is ready, the data is displayed.

As the column is an additional column, the customer won't have the possibility to filter / sort with this column.

I made some mockups for example:
image
image
image

Where do you expect to use this?

ips uapp

Do you have mock-up?

No response

When do you expect this to be delivered?

No response

Additional Information

No response

@aboungnaseng-ovhcloud
Copy link
Contributor

aboungnaseng-ovhcloud commented Jan 16, 2025

So to tackle this ticket we need to propritize the column visibilibily first 👍 #14244

Otherwise, we need to define all use scenarios, in case of loading, in case of error etc ...

@anooparveti
Copy link
Contributor

Deprioritising the ticket since a work-around has been found by @qpavy and moving the ticket to "On Hold".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants