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

Form $attempted status #5574

Closed
wants to merge 2 commits into from
Closed

Conversation

nicoabie
Copy link
Contributor

A contribution to issue #583

Added new state to form:
- it sets to true when form submitted having $invalid state
- it sets back to false when form submitted having valid state
@ghost ghost assigned tbosch and matsko Dec 31, 2013
@IgorMinar
Copy link
Contributor

I think that 'attempted' is not the best name, can we think of something better?

@nicoabie
Copy link
Contributor Author

nicoabie commented Jan 3, 2014

Hi @IgorMinar, thanks for your reply.
The other adjective I can think of is 'rejected'.

@troch
Copy link

troch commented Jan 19, 2014

@IgorMinar @nicoabie exposed?

@nicoabie
Copy link
Contributor Author

@troch It could be.

@matsko
Copy link
Contributor

matsko commented Jun 3, 2014

For AngularDart we use $submitted.

@matsko
Copy link
Contributor

matsko commented Jul 1, 2014

@nicoabie can you squash both commits together and use $submitted instead of attempted?

Also, when $setPristine is called on the form, it should revert the $submitted back to false.

@nicoabie
Copy link
Contributor Author

nicoabie commented Jul 2, 2014

@matsko will do it tonight, Argentina time. thanks for the clarification.

@matsko
Copy link
Contributor

matsko commented Jul 2, 2014

Excellent :)

@nicoabie
Copy link
Contributor Author

nicoabie commented Jul 3, 2014

@matsko I opened a new pull request, need to get better at git commands.

#8056

@nicoabie nicoabie closed this Jul 3, 2014
@nicoabie nicoabie deleted the form_attempted branch July 3, 2014 03:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants