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

[MDS] Data connection details and actions implementation #2

Closed
wants to merge 50 commits into from
Closed

[MDS] Data connection details and actions implementation #2

wants to merge 50 commits into from

Conversation

RyanL1997
Copy link
Owner

Signed-off-by: Ryan Liang [email protected]### Description

Issues Resolved

Screenshot

Testing the changes

Changelog

Check List

  • All tests pass
    • yarn test:jest
    • yarn test:jest_integration
  • New functionality includes testing.
  • New functionality has been documented.
  • Update CHANGELOG.md
  • Commits are signed per the DCO using --signoff
    Signed-off-by: Ryan Liang [email protected]

@RyanL1997 RyanL1997 force-pushed the unified-datasources-2 branch 3 times, most recently from 77b3d5e to 8024f6f Compare July 19, 2024 00:44
RyanL1997 added 26 commits July 18, 2024 22:37
@RyanL1997
Data connection details draft - 0
23c3963 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Data connection details draft 1 - Add datasource overview section
1790680 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Add access control tab draft 0 - add the tab of ac
eb99439 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Add access control tab draft 1 - add the ac as a tab
2233322 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Cache manager draft 0 - setup for with a testable acceleration table
5def0a8 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Cache manager draft 1 - fix the acceleration name
3e1e678 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Cache manager draft 2 - fix the acceleration status color
17f2d7a 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Cache manager draft 3 - Add action menu and action overlay
880b5ee 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Acceleration draft 0 - Implement acceleration actions
396fee7 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Acceleration draft 1 - Fix the table title
460ab01 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Acceleration draft 2 - still having issue on the settings api call
6cc8ac5 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Acceleration draft 3 - fix the fetching apis and still having issue o…
05ef046 
…n the notiffication of flyout operation

Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Acceleration draft 4 - fix the flyout operation
8f0d3fa 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Fix add mds for acceleration flow draft 0 - need to pass mds id into …
0bd8fc9 
…detail page call

Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Add mds for acceleration flow draft 1 - add feature flag check and pa…
13c63b6 
…ss in id for fetch select datasource

Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Add mds for acceleration flow draft 2 - add mds flow to all the accel…
c31b1fe 
…eration table action flow

Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Add mds for acceleration flow draft 3 - remove unused variable in acc…
0e2e2c3 
…eleration table class

Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Fix the route for datasource detail
99b92ff 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Add mds for acceleration flow draft 4 - add mds support for accelerat…
14947ce 
…ion operations table triggered flow

Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Add mds for acceleration flow draft 5 - add mds support for accelerat…
11a4ada 
…ion operation flyout trigger flow

Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Add acceleration creation flow draft 0 - directly migrate the entire …
235b296 
…acc create flow

Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Add acceleration creation flow draft 1 - relocate the flyout render i…
470e405 
…nto plugin ts

Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Add acceleration creation flow draft 2 - fix the query work bench check
d629ed2 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Add acceleration creation flow draft 3 - fix the acceleration creatio…
1d41f51 
…n flow when mds is disabled 0

Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Add acceleration creation flow draft 4 - unrelated to the creation fl…
d3c8c9e 
…ow but add the cache hooks for all the meta info

Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Add acceleration creation flow draft 5 - mds support with acceleratio…
1e4dd06 
…n creation flow

Signed-off-by: Ryan Liang <[email protected]>
RyanL1997 added 22 commits July 18, 2024 22:37
@RyanL1997
Add associated objects flow draft 0 - works without adding mds support
ae01037 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Remove some comments 0
8d3dee4 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Add associated objects flow draft 1 - fix the navi to app
d73da46 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Add associated objects flow draft 2 - fix the navi to app and add mds…
200a1dd 
… support 0

Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
unit test 0 - add unit test for ao utils
9d919d4 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
unit test 1 - setup utils dir for connection detail and add unit tests
0a31730 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
unit test 2 - Add uniti test for acc flyout modules 0
4df2a00 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Enable card on connection details for query in discover
0b23b8c 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Add the mds support for acc flyout details fetch functions
e6ad388 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
unit test 3 - Add unit test for plugin ts flyout setup
1671f18 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
unit test 4 - Add unit test for some of the acceleration components
f3ea66c 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
unit test 5 - clean up the operation unit test a bit
ea54f9e 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Implement the acceleration flyout forwarding with mds support
beae9a5 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Add discover forwarding action
b4468c5 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
unit test 6 - add more test cases on accelerations comoponents
7ae72b1 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
IMPORTANT - fix mds flows for cashloader when mds enabled
68efaf1 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
IMPORTANT - fix mds flows for direct query when mds enabled
e0d8b13 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Fix acc details tab tests
03af2de 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Fix everything in mds refresh
fe0d163 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
P00000000 - revert the change of cacheloader and correct the cacheloa…
0b0c47d 
…der and cachemnger usage

Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Uncomment security role check
14ffdaf 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997
Fix unit tests snapshot
c7fb72d 
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997 RyanL1997 force-pushed the unified-datasources-2 branch from 23ea1e4 to c7fb72d Compare July 19, 2024 05:50
@RyanL1997 RyanL1997 force-pushed the unified-datasources-2 branch from 9322336 to 3e1715e Compare July 19, 2024 17:13
@RyanL1997 @ps48
Add the server setup
e863cbf 
Co-authored-by: Shenoy Pratik <[email protected]>
Signed-off-by: Ryan Liang <[email protected]>
@RyanL1997 RyanL1997 force-pushed the unified-datasources-2 branch from 3e1715e to e863cbf Compare July 19, 2024 17:26
@RyanL1997 RyanL1997 closed this Jul 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
first-time-contributor
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@RyanL1997