Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Require authentication for the settings page #327

Closed
IgorDePaula opened this issue Jul 10, 2014 · 5 comments
Closed

Require authentication for the settings page #327

IgorDePaula opened this issue Jul 10, 2014 · 5 comments
Labels

Comments

@IgorDePaula
Copy link

I access route settings without i am loggin.

@remicastaing
Copy link
Contributor

You're right. In client/app/account/account.js, you should find this:

.state('settings', {
        url: '/settings',
        templateUrl: 'app/account/settings/settings.html',
        controller: 'SettingsCtrl',
        authenticate: true
      });

authenticate: true is missing.

@IgorDePaula
Copy link
Author

yes, now work fine, thanks.

@remicastaing
Copy link
Contributor

@IgorDePaula, you shouldn't close this issue. It has to be fixed.

@JaKXz JaKXz added duplicate and removed duplicate labels Jul 10, 2014
@IgorDePaula
Copy link
Author

i dont understand.

@JaKXz
Copy link
Collaborator

JaKXz commented Jul 10, 2014

This issue was alluded to previously in #305, and @remicastaing I agree it should be fixed for the settings page.

@IgorDePaula IgorDePaula reopened this Jul 10, 2014
@JaKXz JaKXz changed the title Auth dont work Require authentication for the settings page Jul 10, 2014
@JaKXz JaKXz added bug and removed duplicate labels Jul 10, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants