Skip to content

Inconsistency in ui, docs and python ui_server #3497

@bjakubowskiVNSNY

Description

@bjakubowskiVNSNY

Expected Behavior

I would expect that the react app, python server, and docs are consistent in how they implement or (for the docs) describe how the registry is ingested.

Current Behavior

Right now:

  1. Docs: The docs suggest that the UI loads a JSON dump of the registry
  2. Python App: The python app appears to use get_registry_dump to produce a JSON dump of the registry, but get_registry_dump is never called. Instead the UI fetches the registry_proto from /registry (see here and here).
  3. UI: useLoadRegistry looks like it used to parse json -- as in feat: Initial merge of Web UI logic #2352 -- but then chore: Replace zod by protobufjs #3208 replaced the json with the protobuf registry

Steps to reproduce

NA

Specifications

  • Version: main

Possible Solution

If the intent of #3208 was to replace the json registry with the protobuf version, then the solution would be to:

  1. Update the docs to make this clear
  2. Update the python ui_server so it doesn't appear to produce the JSON dump via get_registry_dump

Of these solutions, the first (updating docs) is probably the most valuable to the community.

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions