-
Notifications
You must be signed in to change notification settings - Fork 199
feat: Add on-demand federated API discovery UI with async task management and result rendering #1383
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
Open
DinithEdirisinghe
wants to merge
12
commits into
wso2:manual-federation-feature
Choose a base branch
from
DinithEdirisinghe:main
base: manual-federation-feature
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
feat: Add on-demand federated API discovery UI with async task management and result rendering #1383
Changes from all commits
Commits
Show all changes
12 commits
Select commit
Hold shift + click to select a range
1de1095
feat: add landing page components and localized strings for publisher…
DinithEdirisinghe 6b49108
Async implementation
DinithEdirisinghe 32bc2bf
can switch between schedular and on demand but still the disovery bu…
DinithEdirisinghe 0c80697
Separate UI Results by Gateway,Categorize Gateways in UI
DinithEdirisinghe 18372e1
search option, pagination, retry option
DinithEdirisinghe 3e230d5
feat: implement DiscoverAPIs component with federated API discovery a…
DinithEdirisinghe 0c6cecc
feat: implement federated API discovery result viewer with polling an…
DinithEdirisinghe 70aacc6
feat: implement federated API discovery module and integration in the…
DinithEdirisinghe c43989c
revert: restore npm ci in publisher pom.xml (Windows-specific workaro…
DinithEdirisinghe 929d45b
Fixing SonarClould issues
DinithEdirisinghe 4a96331
feat: add DiscoverAPIsCard component to publisher landing page
DinithEdirisinghe 94a21f3
sonarcloud fixes
DinithEdirisinghe File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -21,9 +21,12 @@ import { Route, Switch, Redirect } from 'react-router-dom'; | |
| import Progress from 'AppComponents/Shared/Progress'; | ||
| import AuthManager from 'AppData/AuthManager'; | ||
| import MCPRouteGuard from 'AppComponents/Shared/MCPRouteGuard'; | ||
| import FederatedAPIDiscoveryRouteGuard from 'AppComponents/Shared/FederatedAPIDiscoveryRouteGuard'; | ||
|
|
||
| import Listing from './Listing/Listing'; | ||
| import APICreateWithAI from './Create/CreateAPIWithAI/APICreateWithAI'; | ||
| import DiscoverAPIs from './Discover/DiscoverAPIs'; | ||
| import DiscoveryResults from './Discover/DiscoveryResults'; | ||
|
|
||
| /* if needs to pre fetch use 'webpackPrefetch: true' */ | ||
|
|
||
|
|
@@ -58,21 +61,21 @@ const Apis = () => { | |
| <Route | ||
| exact | ||
| path='/apis' | ||
| key={Date.now()} | ||
| key='apis' | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. any particular reason for this change? |
||
| render={(props) => <Listing {...props} isAPIProduct={false} />} | ||
| /> | ||
| <Route | ||
| exact | ||
| path='/api-products' | ||
| key={Date.now()} | ||
| key='api-products' | ||
| render={(props) => { | ||
| return <Listing {...props} isAPIProduct />; | ||
| }} | ||
| /> | ||
| <Route | ||
| exact | ||
| path='/mcp-servers' | ||
| key={Date.now()} | ||
| key='mcp-servers' | ||
| render={(props) => ( | ||
| <MCPRouteGuard> | ||
| <Listing {...props} isMCPServer /> | ||
|
|
@@ -82,9 +85,30 @@ const Apis = () => { | |
| <Route | ||
| exact | ||
| path='/search' | ||
| key={Date.now()} | ||
| key='search' | ||
| render={(props) => <Listing {...props} isAPIProduct={false} />} | ||
| /> | ||
| <Route | ||
| exact | ||
| path='/apis/discover' | ||
| key='apis-discover' | ||
| render={(props) => ( | ||
| <FederatedAPIDiscoveryRouteGuard> | ||
| <DiscoverAPIs {...props} /> | ||
| </FederatedAPIDiscoveryRouteGuard> | ||
| )} | ||
| /> | ||
| <Route | ||
| exact | ||
| path='/apis/discover/apis' | ||
| key='apis-discover-apis' | ||
| render={(props) => ( | ||
| <FederatedAPIDiscoveryRouteGuard> | ||
| <DiscoveryResults {...props} /> | ||
| </FederatedAPIDiscoveryRouteGuard> | ||
| )} | ||
| /> | ||
|
|
||
| <Route path='/apis/create' component={DeferredAPICreateRoutes} /> | ||
| <Route path='/apis/design-assistant' component={DefferedAIApiCreateRoutes} /> | ||
| <Route | ||
|
|
@@ -97,8 +121,8 @@ const Apis = () => { | |
| } | ||
| }} | ||
| /> | ||
| <Route | ||
| path='/mcp-servers/create' | ||
| <Route | ||
| path='/mcp-servers/create' | ||
| render={(props) => ( | ||
| <MCPRouteGuard> | ||
| <DeferredAPICreateRoutes {...props} /> | ||
|
|
||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
What is the reason for this script change?