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

使用者角色與權限 CURD 部分 #66

Merged
merged 8 commits into from Jun 5, 2019

Conversation

puckwang
Copy link
Member

@puckwang puckwang commented Jun 3, 2019

主要內容

@puckwang puckwang requested a review from a team June 3, 2019 18:29
@puckwang puckwang self-assigned this Jun 3, 2019
Copy link
Collaborator

@hashman hashman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

先提幾個問題好了,你們幫我查一下

  1. php artisan migrate:refresh --seed 後我用 admin 登入沒有權限可以看到 telegram 相關功能與使用者管理系列
  2. 使用者管理 -> 角色管理,跳進去會馬上回到首頁
  3. 如果是 cache 影響 1 的話,那增加一個 artisan command 來 refresh 所有的權限

@puckwang
Copy link
Member Author

puckwang commented Jun 4, 2019

先提幾個問題好了,你們幫我查一下

  1. php artisan migrate:refresh --seed 後我用 admin 登入沒有權限可以看到 telegram 相關功能與使用者管理系列
  2. 使用者管理 -> 角色管理,跳進去會馬上回到首頁
  3. 如果是 cache 影響 1 的話,那增加一個 artisan command 來 refresh 所有的權限

這有點詭異的,我這邊測不出這幾個錯誤.....

@unciax 你能試試看有沒有相關問題嗎?

Copy link
Collaborator

@hashman hashman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@hashman
Copy link
Collaborator

hashman commented Jun 4, 2019

@puckwang rebase 一下好了,謝謝

@unciax
Copy link
Contributor

unciax commented Jun 4, 2019

先提幾個問題好了,你們幫我查一下

1. `php artisan migrate:refresh --seed` 後我用 admin 登入沒有權限可以看到 telegram 相關功能與使用者管理系列

2. 使用者管理 -> 角色管理,跳進去會馬上回到首頁

3. 如果是 cache 影響 1 的話,那增加一個 artisan command 來 refresh 所有的權限

@hashman
我想我這邊有重現出來 2 的問題了,想確認一下你重新整理之後,直接點選會不會跳回首頁?

@puckwang puckwang force-pushed the feature/55-user-role-acl-crud branch from c677365 to 97262b2 Compare June 4, 2019 15:50
@puckwang
Copy link
Member Author

puckwang commented Jun 4, 2019

先提幾個問題好了,你們幫我查一下

1. `php artisan migrate:refresh --seed` 後我用 admin 登入沒有權限可以看到 telegram 相關功能與使用者管理系列

2. 使用者管理 -> 角色管理,跳進去會馬上回到首頁

3. 如果是 cache 影響 1 的話,那增加一個 artisan command 來 refresh 所有的權限

@hashman
我想我這邊有重現出來 2 的問題了,想確認一下你重新整理之後,直接點選會不會跳回首頁?

我修正了上面 2 的錯誤以及 @unciax 提出的那個問題。

@puckwang puckwang merged commit 1715950 into develop Jun 5, 2019
@puckwang puckwang deleted the feature/55-user-role-acl-crud branch June 5, 2019 03:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants