Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

docs(guide/Migrating from Previous Versions): add info for 1.4 ngPatt… #15765

Merged
merged 2 commits into from
Mar 7, 2017

Conversation

diegomrsantos
Copy link
Contributor

…ern and pattern breaking change

Introduced in commit 0e00108

Closes #15758

…ern and pattern breaking change

Introduced in commit 0e00108

Closes angular#15758
@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
  • If you signed the CLA as a corporation, please let us know the company's name.

1 similar comment
@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
  • If you signed the CLA as a corporation, please let us know the company's name.

@diegomrsantos
Copy link
Contributor Author

I signed it!

@googlebot
Copy link

CLAs look good, thanks!

1 similar comment
@googlebot
Copy link

CLAs look good, thanks!


Due to [0e001084](https://github.com/angular/angular.js/commit/0e001084ffff8674efad289d37cb16cc4e46b50a),
The `ngPattern` and `pattern` directives will validate the regex
against the `viewValue` of `ngModel`, i.e. the value of the model
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You should prefix viewValue, modelValue with $ as this is how they are stored in the model

@Narretz
Copy link
Contributor

Narretz commented Mar 1, 2017

Can you add a note to the commit message that the content in the migration guide is taken from the commit message of 0e00108?

…delValue with $ as this is how they are stored in the model

This content being included in the migration guide is taken from the commit message of 0e00108
@Narretz Narretz merged commit b779091 into angular:master Mar 7, 2017
Narretz pushed a commit that referenced this pull request Mar 7, 2017
Breaking change was introduced in commit 0e00108.
This content being included in the migration guide is taken from the commit message of commit 0e00108.

Closes #15758
Closes #15765
ellimist pushed a commit to ellimist/angular.js that referenced this pull request Mar 15, 2017
Breaking change was introduced in commit 0e00108.
This content being included in the migration guide is taken from the commit message of commit 0e00108.

Closes angular#15758
Closes angular#15765
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants