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

:ambulance: errors on some actions of attendance manager #739

Conversation

Projects
None yet
3 participants
@KolushovAlexandr
Copy link
Member

commented Apr 17, 2019

No description provided.

* ``Manager`` like Officer, but also has access to *Configuration* menu
* ``Manager`` may also delete partners attendances, has access to *Partners*, *Reports* menus and *Kiosk Mode*

* In order to get access to ``Configuration`` menu user has to have **Administration** ``Settings`` rights

This comment has been minimized.

Copy link
@yelizariev

yelizariev Apr 18, 2019

Member

Can we remove this requirement? Move attendance menu to another place maybe?

This comment has been minimized.

Copy link
@KolushovAlexandr

KolushovAlexandr Apr 23, 2019

Author Member

@yelizariev Config menu is the res.config.settings model and has it's own security configurations.

@KolushovAlexandr KolushovAlexandr force-pushed the KolushovAlexandr:11.0-base_attendance-security_issues branch from 81fcf2a to cfd04b2 Apr 25, 2019

@yelizariev yelizariev merged commit 2426080 into it-projects-llc:11.0 Apr 29, 2019

7 checks passed

Hound No violations found. Woof!
Travis CI - Pull Request Build Passed
Details
ci/branches Branch names are correct
Details
ci/runbot runbot build 31368-739-cfd04b (runtime 751s)
Details
codecov/patch/backend Coverage not affected when comparing 9ae3175...cfd04b2
Details
codecov/patch/tests Coverage not affected when comparing 9ae3175...cfd04b2
Details
coverage/coveralls Coverage remained the same at 68.222%
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.