Skip to content
This repository has been archived by the owner on Jul 1, 2020. It is now read-only.

Added norwegian translation and changes to allow user to remove invalid validators #19

Merged
merged 9 commits into from
Apr 8, 2015

Conversation

tlastad
Copy link
Contributor

@tlastad tlastad commented Apr 7, 2015

Added norwegian translation file

tlastad added 8 commits March 31, 2015 09:00
Easter holiday, cant keep this code locked up in the office for a
week...
Added norwegian translation, and added a new key:
INVALID_REQUIREDSELECTION.
French and spanish have not been translated for new key.
Added norwegian language translation
Fixes for removing invalid validation re. wizard
@tlastad tlastad changed the title Added norwegian translation Added norwegian translation and changes to allow user to remove invalid validators Apr 7, 2015
@tlastad
Copy link
Contributor Author

tlastad commented Apr 7, 2015

Ok, this pull request got a little more than I thought :)

  1. Norwegian translation added
  2. I made a fix in updateErrorMsg so that validatorAttrs were set when validator comes from directive.
  3. Added functionality so that when user presses "previous page" in wizard, all items in validationSummary are removed, and also all matching validators in formElements list. These validators will be re-added automatically when user returns to the page.

If this isn't what you want, let me know :)

If validationError has been mistyped, the code will not find requested
element.
It will then just append a new element as is standard functionality.
But it will never try to find the added element if it revalidates, and
add a new message for each validation. This should fix that problem
ghiscoding added a commit that referenced this pull request Apr 8, 2015
Added norwegian translation and changes to allow user to remove invalid validators
@ghiscoding ghiscoding merged commit 02aa250 into ghiscoding:master Apr 8, 2015
@ghiscoding
Copy link
Owner

Looks all good, I tried the Directive and Service and everything seems fine. I bumped the revision to v1.3.14 in case you use Bower :)

How is the rest working for you? Hopefully it's more stable now, this project is becoming much bigger than I imagined but I think it's worth it, there is a lot of nice features now :)

Thanks for the translation as well, always nice to see different nationalities can use this great project.

Oh and did you have a chance to try the alternate text on validators? How do you like it?

@tlastad
Copy link
Contributor Author

tlastad commented Apr 8, 2015

It all works good.
Alternative text works terriffic. Just like I want it to :)

We're really happy with it :)

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

Successfully merging this pull request may close these issues.

2 participants