FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix(forms): default to unsetValue for value accessors by sis0k0 · Pull Request #846 · NativeScript/nativescript-angular · GitHub

fix(forms): default to unsetValue for value accessors - #846

Merged
vakrilov merged 1 commit into
masterfrom
vlaeva/unset-value
Oct 11, 2017
Merged

vakrilov merged 1 commit into
masterfrom
vlaeva/unset-value

Conversation

sis0k0 commented Jun 14, 2017

Copy link
Copy Markdown
Contributor

Copy link
Copy Markdown
Contributor

Is it possible to add a unit test for this?
Overall [ng-model]/value-accessor seem to break from time to time. Let's try to make a proper example which can be tested with e2e tests and can be used as reference for clients.

sis0k0 force-pushed the vlaeva/unset-value branch from cadbbef to cd189de Compare June 30, 2017 10:26
NativeScript deleted a comment from SvetoslavTsenov Jun 30, 2017
NativeScript deleted a comment from SvetoslavTsenov Jun 30, 2017
sis0k0 self-assigned this Jul 5, 2017
sis0k0 added this to the 3.2 TBD milestone Jul 5, 2017

Copy link
Copy Markdown
Contributor

run ci

ghost added the in progress label Sep 28, 2017

sis0k0 commented Sep 28, 2017

Copy link
Copy Markdown
Contributor Author

uitests

ghost assigned vakrilov Oct 10, 2017

Copy link
Copy Markdown
Contributor

Resolves #943

Copy link
Copy Markdown
Contributor

Rebased onto master

vchimev commented Oct 11, 2017

Copy link
Copy Markdown
Contributor

sdkwebpack

vakrilov merged commit 6940955 into master Oct 11, 2017
vakrilov deleted the vlaeva/unset-value branch October 11, 2017 08:50
ghost removed the in progress label Oct 11, 2017
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

4 participants


Back | FazBrowse Home | New Git URL