Skip to content

Fixing #24, #25 and #30 #27

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

Open
wants to merge 5 commits into
base: feature/refactor
Choose a base branch
from

Conversation

S2MBO
Copy link

@S2MBO S2MBO commented Mar 19, 2015

These commit should fix #24 and #25.

S2MBO added 2 commits March 19, 2015 10:41
…trings to mustache.

Integer zeros would be evaluated to false and not rendered by mustache.
@netbe
Copy link
Contributor

netbe commented Mar 19, 2015

@S2MBO don't we have a testcase to check that fix?

@S2MBO
Copy link
Author

S2MBO commented Mar 19, 2015

I added a testcase for #24.

@netbe Could you please write a test case for #25? Limitations for date and data seem not to be implemented in the iOS template at all so those have to be updated anyway and I do not have the ObjectiveC skills to properly do this.

@netbe
Copy link
Contributor

netbe commented Mar 20, 2015

@S2MBO i am on it

@S2MBO S2MBO changed the title Fixing #24 and #25 Fixing #24, #25 and #30 Apr 8, 2015
@S2MBO
Copy link
Author

S2MBO commented Apr 8, 2015

I found another issue and fixed it. The fix also has a test case for #30.

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.

2 participants