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

Forms API #3475

Merged
merged 2 commits into from Oct 22, 2015
Merged

Forms API #3475

merged 2 commits into from Oct 22, 2015

Conversation

tomchristie
Copy link
Member

@tomchristie tomchristie commented Oct 6, 2015

Closes #2138.
Closes #2060.

@tomchristie tomchristie added this to the 3.3.0 Release milestone Oct 6, 2015
@auvipy
Copy link
Contributor

auvipy commented Oct 6, 2015

looks good

@carltongibson
Copy link
Collaborator

carltongibson commented Oct 7, 2015

@tomchristie Did you want any review on this? Looks good. Needs tests. :-)

@tomchristie
Copy link
Member Author

tomchristie commented Oct 8, 2015

Indeed tests and more comprehensive docs incoming.

@xordoquy
Copy link
Collaborator

xordoquy commented Oct 8, 2015

\o/

@tomchristie
Copy link
Member Author

tomchristie commented Oct 12, 2015

Docs updated. Still to do:

  • Tests.
  • Docs & tests for nested forms.

@carltongibson
Copy link
Collaborator

carltongibson commented Oct 13, 2015

This looks great.

@auvipy auvipy mentioned this pull request Oct 17, 2015
@tomchristie
Copy link
Member Author

tomchristie commented Oct 22, 2015

Given that releasing 3.3.0 and launching the funding campaign is so important (Ie start getting resource onto the project against) I'm going to merge this in its current state.

tomchristie added a commit that referenced this pull request Oct 22, 2015
@tomchristie tomchristie merged commit 6305ae8 into master Oct 22, 2015
@tomchristie tomchristie deleted the forms-api branch Oct 22, 2015
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.

None yet

4 participants