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

ExcludePlacesContext and ExcludeSituationsContext #83

Merged
merged 4 commits into from
Aug 24, 2020

Conversation

ue71603
Copy link
Contributor

@ue71603 ue71603 commented May 23, 2020

Reduce the amount of traffic, if parts of the Context are not needed.

Reduce the amount of traffic, if parts of the Context are not needed.
Copy link
Contributor

@pvgrumbkow pvgrumbkow left a comment

Choose a reason for hiding this comment

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

Please change three things:
a) The switches have effect on the content included in the response. Therefore they should be part of the TripContentFilterGroup.
b) The switches in the TripContentFilterGroup are always include switches. So please define them as IncludeXXX with the default set to true.
c) Please consider also the other OJP services. Where appropriate these switches should also be included into the content filter groups of those requests.

ue71603 and others added 2 commits May 31, 2020 15:15
Please change three things:
a) The switches have effect on the content included in the response. Therefore they should be part of the TripContentFilterGroup.
b) The switches in the TripContentFilterGroup are always include switches. So please define them as IncludeXXX with the default set to true.
c) Please consider also the other OJP services. Where appropriate these switches should also be included into the content filter groups of those requests.
Copy link
Contributor

@pvgrumbkow pvgrumbkow left a comment

Choose a reason for hiding this comment

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

Thanks for the changes. One last (minor) thing: Please remove .gitignore from the commit.

@pvgrumbkow pvgrumbkow added this to the v1.1 milestone Jun 3, 2020
@pvgrumbkow pvgrumbkow merged commit b1251c2 into changes_for_v1.1 Aug 24, 2020
@pvgrumbkow pvgrumbkow deleted the ExcludeContext4 branch August 24, 2020 09:20
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.

3 participants