Skip to content

Fix startDate spelling & clickAnalytics value; handle empty results#1

Merged
pgrimaud merged 1 commit intopgrimaud:masterfrom
pdbreen:repair
Oct 25, 2018
Merged

Fix startDate spelling & clickAnalytics value; handle empty results#1
pgrimaud merged 1 commit intopgrimaud:masterfrom
pdbreen:repair

Conversation

@pdbreen
Copy link
Copy Markdown
Contributor

@pdbreen pdbreen commented Oct 25, 2018

If clickAnalytics=1 parameter is ignored - needs to be clickAnalytics=true

@coveralls
Copy link
Copy Markdown

coveralls commented Oct 25, 2018

Pull Request Test Coverage Report for Build 157

  • 10 of 10 (100.0%) changed or added relevant lines in 4 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 5: 0.0%
Covered Lines: 226
Relevant Lines: 226

💛 - Coveralls

@pgrimaud pgrimaud merged commit 798da19 into pgrimaud:master Oct 25, 2018
@pgrimaud
Copy link
Copy Markdown
Owner

@pdbreen Thanks !

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