-
Notifications
You must be signed in to change notification settings - Fork 9
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
Add methods to provide list of events to back-office #31
Conversation
Pull Request Test Coverage Report for Build 377
💛 - Coveralls |
6059025
to
3f3da13
Compare
8fb09c9
to
2906f0a
Compare
Message: fmt.Sprintf("Duplicated parameter %s", key), | ||
} | ||
} | ||
request[key] = value |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For realm key, we will override the one provided by the path parameter.
Maybe rename it target_realm would solve the issue.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ça serait bien mettre à jour le README
3362add
to
ae50e0c
Compare
…nts to back-office
… of events to back-office
@@ -31,7 +31,7 @@ func MakeGetEventsSummaryEndpoint(ec EventsComponent) endpoint.Endpoint { | |||
// MakeGetUserEventsEndpoint makes the events summary endpoint. | |||
func MakeGetUserEventsEndpoint(ec EventsComponent) endpoint.Endpoint { | |||
return func(ctx context.Context, req interface{}) (interface{}, error) { | |||
params := filterParameters(req.(map[string]string), "first", "max", "dateFrom", "dateTo", "realm", "userID", "origin", "ctEventType") | |||
params := filterParameters(req.(map[string]string), "first", "max", "dateFrom", "dateTo", "realm", "realmTarget", "userID", "origin", "ctEventType") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On devrait pas enelver realm ici ?
No description provided.