search

rydurham / L4withSentry

Laravel 4 and Sentry 2.0
300 stars 91 forks source link

missing register button in index #36

Closed jack6800 closed 10 years ago

jack6800 commented 10 years ago

Hi Thank for your efforts, there is a minor issue in index, missing user register button though there is getRegister in user controller already.

rydurham commented 10 years ago

There is a 'Register' link in the default view, but it is only visible if you are not logged in.

Are you using the most up to date version of the UserController? The 'getRegister' method has been removed. Instead, the register action is handled by the 'create' method of UserController.php

What version of this project are you using?

jack6800 commented 10 years ago

Hi Thank for your timely reply, I'm newbe in Laravel, learning for interesting. I clone your software last month and doing git update just before report these issues. So I assumed all program has been refresh to newest states? Sorry if I "reinvent" fixed issues.

Jack On 2013-12-02 ?? 10:31, Ryan Durham wrote:

There is a 'Register' link in the default view, but it is only visible if you are not logged in.

Are you using the most up to date version of the UserController? The 'getRegister' method has been removed. Instead, the register action is handled by the 'create' method of UserController.php

What version of this project are you using?

Reply to this email directly or view it on GitHub https://github.com/rydurham/L4withSentry/issues/36#issuecomment-29590972.

rydurham commented 10 years ago

No worries! Look at the UserController in your project and compare it to the UserController in the main repo - are they the same? If not try grabbing a fresh copy of the code and see if you run into the same issues.

The problem you were having with the activation link (#38) is similar - check to see if your version of app/views/email/auth/welcome.blade.php is the same as the one in the main repo and see if that resolves the problem.

jack6800 commented 10 years ago

Hi Ryan You are right, what in my clone are old version! I do see a Register button after a fresh installation, thanks. Have no idea why my repository fail to sync/update with yours. Would you kindness enough to point out where went wrong because I really don't want to waste your time and like to avoid such problem in the future. Here is my clone steps:

  1. git clone -b master git://github.com/rydurham/L4withSentry.git l4
  2. CD l4, curl -sS https://getcomposer.org/installer | php
  3. php composer.phar install
  4. git remote rename origin upstream
  5. git remote add origin https://github.com/jack6800/l4.git
  6. git push -u origin master
  7. git fetch upstream
  8. git merge upstream/master
  9. php composer.phar update

I run git remote update before test your software.

No doubt a validation system is a base of all serious web application, a good administrate tools are very important, since your goal seems very clear "A demo of Sentry", so I believe there shall have a back-end tools to manage all kind of users/group and permissions, that is why I add a register button in Users page.

On 2013-12-03 ?? 04:06, Ryan Durham wrote:

No worries! Look at the UserController in your project and compare it to the UserController https://github.com/rydurham/L4withSentry/blob/master/app/controllers/UserController.php in the main repo - are they the same? If not try grabbing a fresh copy of the code and see if you run into the same issues.

The problem you were having with the activation link (#38 https://github.com/rydurham/L4withSentry/issues/38) is similar - check to see if your version of |app/views/email/auth/welcome.blade.php| is the same as the one in the main repo and see if that resolves the problem.

Reply to this email directly or view it on GitHub https://github.com/rydurham/L4withSentry/issues/36#issuecomment-29652536.

rydurham commented 10 years ago

It seems like your procedures should work. I am not sure what happened, but if it is working now you should be all set.

rydurham commented 10 years ago

I am going to close this issue - have you been able to resolve the problem?

jack6800 commented 10 years ago

Yes, thanks On 2013-12-10 ?? 01:55, Ryan Durham wrote:

I am going to close this issue - have you been able to resolve the problem?

Reply to this email directly or view it on GitHub https://github.com/rydurham/L4withSentry/issues/36#issuecomment-30154981.