Skip to content

Conversation

@wojciechkedziora
Copy link
Contributor

Hi,

Previously entity graph passed to streaming methods as separate parameter in repository classes (extended with EntityGraphRepository) is not applied. This change contains mentioned fix + additional streaming repository tests.

Thanks.

@wojciechkedziora wojciechkedziora force-pushed the add_support_for_entity_graphs_in_streaming_methods branch from c4fb37f to 6a5742d Compare February 10, 2023 14:37
@wojciechkedziora wojciechkedziora force-pushed the add_support_for_entity_graphs_in_streaming_methods branch from 6a5742d to e7f3960 Compare February 13, 2023 15:12
@reda-alaoui
Copy link
Member

Please rebase the PR on master to trigger the CI job that I just configured on master.

@wojciechkedziora wojciechkedziora force-pushed the add_support_for_entity_graphs_in_streaming_methods branch from e7f3960 to 659bdea Compare February 14, 2023 07:57
Copy link
Member

@reda-alaoui reda-alaoui left a comment

Choose a reason for hiding this comment

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

Please fix the code formatting with ./mvnw git-code-format:format-code -Dgcf.globPattern=**/*

@wojciechkedziora wojciechkedziora force-pushed the add_support_for_entity_graphs_in_streaming_methods branch from 659bdea to 5b44b89 Compare February 14, 2023 17:10
@reda-alaoui reda-alaoui merged commit 2dce059 into Cosium:master Feb 14, 2023
@reda-alaoui
Copy link
Member

Thank you @wojciechkedziora !

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