cross-solution / YAWIK

YAWIK is a web application. It can be used as an ATS applicant tracking system or as a jobboard.
https://yawik.org
MIT License
124 stars 67 forks source link

Register twice display the form again with no message #93

Closed mbo-s closed 8 years ago

mbo-s commented 9 years ago

If you register the first time you get an email to confirm your account. If you do not confirm the account, but register again with the same email the form will display again with no error message. Better would be a message: The Email is already registered, Lost your password, resend confirmation mail (if the account is not confirmed)

mbo-s commented 9 years ago

That is no an issue regarding yawik. I used an modified layout. The Javascript for the notify should be loaded after the core.ini.js Javascript Something that should be mentioned in http://yawik.readthedocs.org/en/latest/modules/core/notifications.html

Right order to display the notifications $this->configheadScript() ->prependFile($this->basepath('Core/js/core.init.js')) ->prependFile($this->basePath('/Core/js/pnotify.custom.min.js'));

TiSiE commented 8 years ago

$this->configheadScript() ->prependFile($this->basepath('Core/js/core.init.js')) ->prependFile($this->basePath('/Core/js/pnotify.custom.min.js'));

This will load the notify script BEFORE the core script.

This is right:

$this->configheadScript()
->prependFile($this->basepath('Core/js/core.init.js'))
->appendFile($this->basePath('/Core/js/pnotify.custom.min.js'));

or

$this->configheadScript()
->prependFile($this->basePath('/Core/js/pnotify.custom.min.js'))
->prependFile($this->basepath('Core/js/core.init.js'));