X Tutup
Skip to content

fix(form): Form directives are exportedAs 'ngForm' (was 'form')#5709

Closed
vicb wants to merge 1 commit intoangular:masterfrom
vicb:1208-ngForm
Closed

fix(form): Form directives are exportedAs 'ngForm' (was 'form')#5709
vicb wants to merge 1 commit intoangular:masterfrom
vicb:1208-ngForm

Conversation

@vicb
Copy link
Copy Markdown
Contributor

@vicb vicb commented Dec 8, 2015

fixes #5658

BREAKING CHANGE:

Before:

<form #f="form">

After:

<form #f="ngForm">

fixes angular#5658

BREAKING CHANGE:

Before:

    <form #f="form">

After:

    <form #f="ngForm">
@vicb vicb added action: review The PR is still awaiting reviews from at least one requested reviewer comp: core/forms refactoring Issue that involves refactoring or code-cleanup labels Dec 8, 2015
@vsavkin vsavkin added pr_state: LGTM action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Dec 8, 2015
@vsavkin vsavkin assigned vicb and unassigned vsavkin Dec 8, 2015
@mary-poppins
Copy link
Copy Markdown

Merging PR #5709 on behalf of @jelbourn to branch presubmit-jelbourn-pr-5709.

@vicb vicb closed this in 8657ca4 Dec 8, 2015
@rolandjitsu
Copy link
Copy Markdown

This did not make it in the changelog. There seems to be some issues with a few things missing from the changelog, could there be something done about that?

@vicb
Copy link
Copy Markdown
Contributor Author

vicb commented Dec 9, 2015

This did not make it in the changelog.

It did. There was some issue with the formatting, there is already a pending PR.

@vicb vicb deleted the 1208-ngForm branch June 23, 2016 17:44
@angular-automatic-lock-bot
Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 8, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker cla: yes refactoring Issue that involves refactoring or code-cleanup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

RFC: rename exportAs: 'form' to exportAs: 'ngForm'

6 participants

X Tutup