Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GraphQL projects output field #1547

Open
wants to merge 3 commits into
base: hotfix-1.10.x
Choose a base branch
from
Open

Conversation

plyhun
Copy link
Contributor

@plyhun plyhun commented Oct 23, 2023

Abstract

Add an ability to output the list of projects, assigned to the (micro)schema. Filterable.

Checklist

General

  • Added abstract that describes the change
  • Added changelog entry to /CHANGELOG.adoc
  • Ensured that the change is covered by tests
  • Ensured that the change is documented in the docs

On API Changes

  • Checked if the changes are breaking or not
  • Added GraphQL API if applicable
  • Added Elasticsearch mapping if applicable

@plyhun plyhun requested a review from npomaroli October 23, 2023 09:23
@@ -146,7 +146,7 @@ public Result<? extends HibNodeFieldContainer> findDraftFieldContainers(HibSchem

@Override
public Result<HibProject> findLinkedProjects(HibSchema schema) {
return new TraversalResult<>(getRoots(schema).stream().map(root -> root.getProject()));
return new TraversalResult<>(getRoots(schema).stream().map(root -> root.getProject()).filter(Objects::nonNull));
Copy link
Contributor Author

Choose a reason for hiding this comment

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

In the test AW MP database there was a case when a ODB edge existed, but lead to nothing.


@Override
protected List<FilterField<HibProject, ?>> getFilters() {
List<FilterField<HibProject, ?>> filters = new ArrayList<>();
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Possible filtering fields for the project: name, UUID, user tracking.

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.

1 participant