Age | Commit message (Collapse) | Author |
|
|
|
|
|
There was a weird case with reloading the user in the test if the public key had been unset.
|
|
|
|
|
|
Conflicts:
users/app/views/users/edit.html.haml
|
|
want to leak ID information.
|
|
|
|
changed the way the presenter works. Will need functional testing
|
|
|
|
|
|
different display.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
Conflicts:
users/app/views/emails/_email.html.haml
|
|
This isn't ready to merge, as there is an issue with logging in as an admin in the test.
|
|
Conflicts:
users/app/controllers/users_controller.rb
|
|
test after merging in show view for users.
|
|
|
|
|
|
This involves a number of other changes like making sure the comparison between aliases and emails still works.
Will do that by removing the @domain.tld from aliases as well.
|
|
is an admin, they should see an alert that the object wasn't found, and be redirected to the current controller.
If the object isn't found and the current user is not an admin, then we will continue to give an error about no access, so as not to leak information about what IDs do and don't exist.
|
|
listing the tickets. Give a message if a user has no tickets.
|
|
|
|
Just found out that render(@collection) returns nil for emtpy collections. So that is usefull for putting messages about the emtpy collection in an or clause.
|
|
partials have specific CSS for another use, so we will likely want to tweak this.
|
|
Conflicts:
users/test/functional/users_controller_test.rb
|
|
|
|
|
|
|
|
|
|
This way we get syntax highlighting and so on.
|
|
validating email domain and displaying it as the placeholder
|
|
|
|
|
|
|
|
This even works client side. :)
|
|
|
|
|
|
using CouchRests user.email_aliases.build so the casted_by method is set in the alias
Used this to move the validations into the alias itself. This is where they belong and allows us to render the errors inline along the email field they belong to.
|
|
still needs a bit of refactoring in the view
|
|
|
|
|
|
|