Skip to content

Comments

Fix form submit validation#2

Merged
pirxpilot merged 1 commit intopirxpilot:nextfrom
tumpio:next
Jul 19, 2017
Merged

Fix form submit validation#2
pirxpilot merged 1 commit intopirxpilot:nextfrom
tumpio:next

Conversation

@tumpio
Copy link

@tumpio tumpio commented Jul 2, 2017

as discussed in #1

@tumpio
Copy link
Author

tumpio commented Jul 2, 2017

I have another pull request coming for editing inserted tags:
tumpio@5c3177c

@pirxpilot
Copy link
Owner

oops - sorry for the delay; I was on vacation and somehow missed a notification on this one. Will try to have a look this weekend. If you find some time please rebuild your branch not to include my merge commit, if not - no worries, I'll do that.

@tumpio
Copy link
Author

tumpio commented Jul 16, 2017

If you find some time please rebuild your branch not to include my merge commit, if not - no worries, I'll do that.

Sure! I spotted one other issue as well. It should not include 'value' property in MOVE_PROPS.

@tumpio
Copy link
Author

tumpio commented Jul 17, 2017

Should be mergeable now.

@pirxpilot pirxpilot merged commit 2713e39 into pirxpilot:next Jul 19, 2017
@pirxpilot
Copy link
Owner

released in 2.1.7 - sorry it took so long

@tumpio
Copy link
Author

tumpio commented Jul 21, 2017

No worries 👍

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