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

BUGFIX: CL-9654 Added measure index mapping to response headers #156

Merged
merged 4 commits into from Jun 29, 2016

Conversation

bredy
Copy link
Contributor

@bredy bredy commented Jun 27, 2016

No description provided.

Jan Pradac added 3 commits June 23, 2016 16:50
… response

- fixed invalid "does not execute all-time date filter" and "does not execute attribute filter with all selected"
- metricMappings are necessary for post processing result headers and letting know which metric is from witch measure bucket and if is PoP or not
@gdgate
Copy link
Contributor

gdgate commented Jun 27, 2016

Build succeeded.

@yenkins-admin
Copy link

Merged build triggered.

@@ -154,6 +154,41 @@ describe('execution', () => {
done();
});
});

it('should wrap response headers with metric mappings', done => {
Copy link
Contributor

Choose a reason for hiding this comment

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

remove done, return promise from test fn

@gdgate
Copy link
Contributor

gdgate commented Jun 29, 2016

Build succeeded.

@akloboucnik akloboucnik merged commit 5bfcc2b into gooddata:master Jun 29, 2016
@bredy bredy deleted the CL-9654 branch June 29, 2016 12:00
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.

None yet

4 participants