Skip to content

Commit

Permalink
fix: Read project data from the 'projects' key while loading the regi…
Browse files Browse the repository at this point in the history
…stry state in the Feast UI (#4772)

* fix: Read project data from the 'projects' key while loading the registry state in the Feast UI

Signed-off-by: boliri <[email protected]>

* fix: regenerate registry file so that UI tests can parse the 'projects' object

Signed-off-by: boliri <[email protected]>

* fix: Check for non-null and non-undefined spec of a project to mitigate errors in UI builds

Signed-off-by: boliri <[email protected]>

---------

Signed-off-by: boliri <[email protected]>
  • Loading branch information
boliri authored Jan 21, 2025
1 parent 6926a7c commit cb81939
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 2 deletions.
Binary file modified ui/public/registry.db
Binary file not shown.
2 changes: 1 addition & 1 deletion ui/src/FeastUISansProviders.test.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ test("full app rendering", async () => {
expect(screen.getByText(/Explore this Project/i)).toBeInTheDocument();

const projectNameRegExp = new RegExp(
parsedRegistry.projectMetadata[0].project!,
parsedRegistry.projects[0].spec?.name!,
"i"
);

Expand Down
2 changes: 1 addition & 1 deletion ui/src/queries/useLoadRegistry.ts
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ const useLoadRegistry = (url: string) => {
// });

return {
project: objects.projectMetadata[0].project!,
project: objects.projects[0].spec?.name!,
objects,
mergedFVMap,
mergedFVList,
Expand Down

0 comments on commit cb81939

Please sign in to comment.