Ditch the feature flag.
Signed-off-by: Eric Peterson <ericpeterson@spotify.com>
This commit is contained in:
@@ -17,7 +17,6 @@
|
||||
import {
|
||||
AlertDisplay,
|
||||
createApp,
|
||||
LocalStorageFeatureFlags,
|
||||
FlatRoutes,
|
||||
OAuthRequestDialog,
|
||||
SignInPage,
|
||||
@@ -89,7 +88,6 @@ const app = createApp({
|
||||
|
||||
const AppProvider = app.getProvider();
|
||||
const AppRouter = app.getRouter();
|
||||
const featureFlags = new LocalStorageFeatureFlags();
|
||||
|
||||
const routes = (
|
||||
<FlatRoutes>
|
||||
@@ -103,9 +101,7 @@ const routes = (
|
||||
</Route>
|
||||
<Route path="/catalog-import" element={<CatalogImportPage />} />
|
||||
<Route path="/docs" element={<TechdocsPage />} />
|
||||
{featureFlags.isActive('use-search-platform') && (
|
||||
<Route path="/search" element={<SearchPage />} />
|
||||
)}
|
||||
<Route path="/search-next" element={<SearchPage />} />
|
||||
<Route path="/create" element={<ScaffolderPage />} />
|
||||
<Route path="/explore" element={<ExplorePage />} />
|
||||
<Route
|
||||
|
||||
@@ -14,7 +14,7 @@ The above search experience is 100% client-side and only looks at the Software C
|
||||
|
||||
In order to work on this new search platform, you will need to be aware of the following:
|
||||
|
||||
- **The use-search-platform feature flag**: The new search platform will move the primary search page to the App-level, out of the search plugin. You'll need to enable this feature flag to work on the new platform.
|
||||
- **In-development app search route**: The new search platform will move the primary search page to the App-level, out of the search plugin. For now, to ensure the old experience is still easy to test, you can work on the new platform at `/search-next` instead of `/search`.
|
||||
- **App SearchPage Component**: You'll find a new search page under `/packages/app/src/components/search`. When ready, we'll add an equivalent page to the `@backstage/create-app` template.
|
||||
- **Backend**: Don't forget, a lot of functionality will be made available in backend plugins:
|
||||
- `@backstage/plugin-search-indexer-backend`, which is responsible for the search index management
|
||||
|
||||
@@ -27,9 +27,8 @@ export const rootRouteRef = createRouteRef({
|
||||
|
||||
export const searchPlugin = createPlugin({
|
||||
id: 'search',
|
||||
register({ router, featureFlags }) {
|
||||
register({ router }) {
|
||||
router.addRoute(rootRouteRef, SearchPageComponent);
|
||||
featureFlags.register('use-search-platform');
|
||||
},
|
||||
routes: {
|
||||
root: rootRouteRef,
|
||||
|
||||
Reference in New Issue
Block a user