Skip to content
This repository was archived by the owner on Jul 29, 2026. It is now read-only.

feat(auth) add confirm password to signup and settings - #483

Closed
thomporter wants to merge 1 commit into
angular-fullstack:canaryfrom
thomporter:canary
Closed

feat(auth) add confirm password to signup and settings#483
thomporter wants to merge 1 commit into
angular-fullstack:canaryfrom
thomporter:canary

Conversation

@thomporter

Copy link
Copy Markdown

also adding for attribute to labels with and ids to associated elements
confirm password powered by directive: https://github.com/wongatech/angular-confirm-field
Files Modified:

app/index.js
app/templates/_bower.json
app/templates/client/app/account(auth)/settings/settings(html).html
app/templates/client/app/account(auth)/settings/settings(jade).jade
app/templates/client/app/account(auth)/signup/signup(html).html
app/templates/client/app/account(auth)/signup/signup(jade).jade
app/templates/karma.conf.js
test/fixtures/bower.json

take note of new dependency in test/fixtures/bower.json (re-run bower install on current clones...)

also adding for attribute to labels with and ids to associated elements
confirm password powered by directive: https://github.com/wongatech/angular-confirm-field
Files Modified:
app/index.js
app/templates/_bower.json
app/templates/client/app/account(auth)/settings/settings(html).html
app/templates/client/app/account(auth)/settings/settings(jade).jade
app/templates/client/app/account(auth)/signup/signup(html).html
app/templates/client/app/account(auth)/signup/signup(jade).jade
app/templates/karma.conf.js
test/fixtures/bower.json

take note of new dependency in test/fixtures/bower.json (re-run bower install on current clones...)
@thomporter

Copy link
Copy Markdown
Author

I swear it was working, but I'm seeing a problem in the HTML. Closing & will re-submit in a moment...

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.

1 participant