- edited description
Missing cases in AccessViewHelper
In the AccessViewHelper
there is lack of case
options for some auth annotations, which causes that class infos table in BE module for access annotation @Api\Access("fe_groups[123]")
displays Access: Not defined! instead of i.e. Access: fe_groups: 123
Missing options:
fe_groups
be_admins
api_users
ip
ip_user
I added them for my own purposes, additionally with human-readable labels (for all access annotations) + different icons for user based and ip based restrictions, so now it looks like:
Access: Frontend user groups [fe_groups
]: 123
screenshots:
Timeless offer: if you’re interested in incorporating this change into the master branch, feel free to let me know, so I’ll fork the repository and will prepare a pull request. :)
Comments (9)
-
reporter -
reporter - edited description
-
reporter - marked as bug
-
Love it! Please create a pull request - that would be great.
-
reporter Bon appetit!
https://bitbucket.org/99grad-team/nnrestapi/pull-requests/6/17-of-original-repo-missing-cases-inBTW, David pls, contact me, please ie. via gmail/hangout, I have several other tweaks, which I’d like to discuss before proposing. biesior
-
reporter ping floyd: is there anybody out there?
-
Howard Carpendale: Hello again!
I tried to contact you via email last week – maybe I had the wrong address.
Here is my mail:Thanks for your commit and bugfix! Looks great. We’ll have it in the next release. Tweaks? Sounds very interesting!If you like we can do a zoom call tomorrow or on Thursday. Let me know what would be a good time for you.
You can contact me here: david@99grad.de
-
- changed status to resolved
Thanks – once again!
-
reporter You’re welcome! Great to know that’s useful! We have still some tweaks to add, I can see the future, it is warm, safe and comfortable, on Bahama islands with palm-drink in the one hand and a glass of Scotch in the second ;)
- Log in to comment