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

Fixed inconsistency #5191

Closed
wants to merge 1 commit into from
Closed

Fixed inconsistency #5191

wants to merge 1 commit into from

Conversation

jperovic
Copy link
Contributor

All examples but "Annotation" use parameter named "username" whereas it uses parameter "name".

All examples but "Annotation" use parameter named  "username" whereas it uses parameter "name".
@xabbuh
Copy link
Member

xabbuh commented Apr 19, 2015

Nice catch @jperovic!

@wouterj
Copy link
Member

wouterj commented Apr 26, 2015

I love the fact that you switched tabs and noticed this inconsistency. Thanks for the PR. I've pushed the commits into the 2.3 branch, which will be merged up to the 2.6 ... master branches. This causes a "closed" status instead of "merged", but the PR is actually merged.

wouterj added a commit that referenced this pull request Apr 26, 2015
This PR was submitted for the 2.6 branch but it was merged into the 2.3 branch instead (closes #5191).

Discussion
----------

Fixed inconsistency

All examples but "Annotation" use parameter named  "username" whereas it uses parameter "name".

Commits
-------

3c9e738 Fixed inconsistency
@wouterj wouterj closed this Apr 26, 2015
@jperovic
Copy link
Contributor Author

Thanks :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants