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.

Sign up for GitHub

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

Jump to bottom

[UI] Data Insight Layout looks broken in case of large number of assets #13803

Closed
mohityadav766 opened this issue Oct 31, 2023 · 2 comments · Fixed by #13921
Closed

[UI] Data Insight Layout looks broken in case of large number of assets #13803

mohityadav766 opened this issue Oct 31, 2023 · 2 comments · Fixed by #13921
Assignees
@ShaileshParmar11
Labels
UI UI specific issues

Comments

@mohityadav766
Copy link
Contributor

Affected module
Does it impact the UI, backend or Ingestion Framework?

Describe the bug
Please see the video attached.

Screen.Recording.2023-10-31.at.11.19.16.PM.mov

To Reproduce

Screenshots or steps to reproduce

Expected behavior
A clear and concise description of what you expected to happen.

Version:

  • OS: [e.g. iOS]
  • Python version:
  • OpenMetadata version: [e.g. 0.8]
  • OpenMetadata Ingestion package version: [e.g. openmetadata-ingestion[docker]==XYZ]

Additional context
Add any other context about the problem here.
@mohityadav766 mohityadav766 added the UI UI specific issues label Oct 31, 2023
mohityadav766 added a commit that referenced this issue Nov 1, 2023
@mohityadav766
Fixes #13803 : Reduce the number of call in case of columns
9f7e17d
@mohityadav766 mohityadav766 mentioned this issue Nov 1, 2023
Merged
9 tasks
@harshach harshach moved this to Data Insights in Release 1.3.0 Nov 4, 2023
mohityadav766 added a commit that referenced this issue Nov 7, 2023
@mohityadav766
Fixes #13803 : Reduce the number of call in case of columns (#13819)
4a73978 
* Fixes #13803 : Reduce the number of call in case of columns

* Add Generic Method and Interface for Fields

* Removed unnecessary comments

* Make FieldInterface return List<? extends FieldInterface> for getChildren

* Fix Failing Tests

* - Remove Pipeline or Kill on app Disable
- Add AppMarketPlaceResourceTest
- Reschedule app automatically on restore

* Add Debug Log in validate Change Event

* Fix BotResourceTest

* Api Refactor on UI
mohityadav766 added a commit that referenced this issue Nov 7, 2023
@mohityadav766
Fixes #13803 : Reduce the number of call in case of columns (#13819)
e91a97e 
* Fixes #13803 : Reduce the number of call in case of columns

* Add Generic Method and Interface for Fields

* Removed unnecessary comments

* Make FieldInterface return List<? extends FieldInterface> for getChildren

* Fix Failing Tests

* - Remove Pipeline or Kill on app Disable
- Add AppMarketPlaceResourceTest
- Reschedule app automatically on restore

* Add Debug Log in validate Change Event

* Fix BotResourceTest

* Api Refactor on UI

(cherry picked from commit 4a73978)
@mohityadav766
Copy link
Contributor Author

Opening this needs work from UI

@mohityadav766 mohityadav766 reopened this Nov 8, 2023
@harshach
Copy link
Collaborator

harshach commented Nov 8, 2023

@chirag-madlani @ShaileshParmar11 where are we with this

ShaileshParmar11 added a commit that referenced this issue Nov 9, 2023
@ShaileshParmar11
[UI] Data Insight Layout looks broken in case of large number of assets
ee4ba43 
#13803
@ShaileshParmar11 ShaileshParmar11 mentioned this issue Nov 9, 2023
Merged
9 tasks
ShaileshParmar11 added a commit that referenced this issue Nov 10, 2023
@ShaileshParmar11
[UI] Data Insight Layout looks broken in case of large number of assets
cefe222 
#13803 (#13921)

* [UI] Data Insight Layout looks broken in case of large number of assets #13803

* move height to constant variable in cost file

* fixed DI cypress

* fixed failing unit test
ShaileshParmar11 added a commit that referenced this issue Nov 10, 2023
@ShaileshParmar11
[UI] Data Insight Layout looks broken in case of large number of assets
e86b578 
#13803 (#13921)

* [UI] Data Insight Layout looks broken in case of large number of assets #13803

* move height to constant variable in cost file

* fixed DI cypress

* fixed failing unit test
ulixius9 pushed a commit that referenced this issue Nov 10, 2023
@ShaileshParmar11 @ulixius9
[UI] Data Insight Layout looks broken in case of large number of assets
54958ca 
#13803 (#13921)

* [UI] Data Insight Layout looks broken in case of large number of assets #13803

* move height to constant variable in cost file

* fixed DI cypress

* fixed failing unit test
@ShaileshParmar11 ShaileshParmar11 moved this from Data Insights to Done in Release 1.3.0 Nov 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ShaileshParmar11 ShaileshParmar11

Labels
UI UI specific issues
Projects
No open projects
Release 1.3.0
Status: Done
Milestone
No milestone
4 participants
@harshach @chirag-madlani @ShaileshParmar11 @mohityadav766