Skip to content

Remove mergedConfigurationSelector and legacy schema-style coupling #1729

@kmcginnes

Description

@kmcginnes

Description

Once all consumers have been migrated (#1726, #1727, #1728), the legacy merge system can be deleted.

Preferred Solution

  • Delete mergedConfigurationSelector and mergeConfiguration()
  • Delete RawConfiguration.schema (the config-provided schema is defunct)
  • Remove the & VertexPreferencesStorageModel intersection from VertexTypeConfig
  • Remove the & EdgePreferencesStorageModel intersection from EdgeTypeConfig
  • Delete any helper functions that only existed to support the 3-way merge (e.g., patchToRemoveDisplayLabel)
  • Update or remove tests that test the old merge behavior

Related Issues


Important

If you are interested in working on this issue, please leave a comment.

Tip

Please use a 👍 reaction to provide a +1/vote. This helps the community and maintainers prioritize this request.

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or requestperformanceIssues relating to performanceschemaIssues related to the schema definition or synchronizationtech debtIssues, typically tasks, that are mainly about cleaning up code that is problematic in some way

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions