Skip to content

SEARCH-444, SEARCH-752, SEARCH-753: Breaking changes to relationship JSON#69

Open
mwiencek wants to merge 8 commits into
metabrainz:masterfrom
mwiencek:relation-fixes
Open

SEARCH-444, SEARCH-752, SEARCH-753: Breaking changes to relationship JSON#69
mwiencek wants to merge 8 commits into
metabrainz:masterfrom
mwiencek:relation-fixes

Conversation

@mwiencek

Copy link
Copy Markdown
Member

This PR is based on top of #68.

Fixes the following issues:

  • SEARCH-444 - Improper relation-list in `area’ and ‘url’ JSON output
  • SEARCH-752 - Relationships have an extra target property in JSON output
  • SEARCH-753 - Missing target-type property in relationships of ‘area’ and ‘url’ JSON output

Since SEARCH-444 changes the relationship list structure for areas and URLs, and SEARCH-752 removes an existing property, these are intended to be announced and released as part of the schema change.

SEARCH-753 is included because it depends on SEARCH-444, and is already being fixed for events and works prior to the schema change in #68.

…d work JSON

The `target-type` is also missing from area and URL relationships, but this
commit only affects event and work relationships. Extending the
`RelationAdapter` usage to areas and URLs would add the `target-type` but also
fix SEARCH-444, which is a larger breaking change.
Adds an adapter based on WorkAdapter.java to flatten the relation lists into a
single `relations` array.

This also fixes SEARCH-753 for areas only.
Adds an adapter based on WorkAdapter.java to flatten the relation lists into a
single `relations` array.

This also fixes SEARCH-753 for URLs.

@reosarevok reosarevok left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTMBDNT

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants