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

ROLE_CIDE_ICZ Data entering control #50

Open
MartinTuchyna opened this issue Dec 29, 2019 · 7 comments
Open

ROLE_CIDE_ICZ Data entering control #50

MartinTuchyna opened this issue Dec 29, 2019 · 7 comments
Labels
out_of_scope Requirements that are not within the CIDE APP specification! testing

Comments

@MartinTuchyna
Copy link
Member

There is no control when SMS criteria are actually finished. Inspector can change it whenever he wants. This should not be possible.

Note 14.6.2019. Information required if it is possible to have information when SMS is finished.

@klimeto
Copy link
Collaborator

klimeto commented Dec 29, 2019

@MartinTuchyna why do you place the ROLE_CIDE_ICZ in the title? Is it an issue linked only to this role? If not remove it from the title!

Inspector can change / add scores until he / she finished the inspection, so naturally he / she can change them whenever.

If you wish to have a locking mechanisms for this, it is out of scope of the CIDE APP specification!

@klimeto klimeto added the out_of_scope Requirements that are not within the CIDE APP specification! label Dec 29, 2019
@MartinTuchyna
Copy link
Member Author

@klimeto
The title ROLE_CIDE_ICZ was copied from the TIS_CIDE_GitHub_final.docx document provided by the testers in order to easily identify the original issue.
Not sure, if it is related only to this role, as I could not test this role. This shall be confirmed by the author of the issue.

@klimeto
Copy link
Collaborator

klimeto commented Dec 30, 2019

@MartinTuchyna

I kindly asked you not only to copy but also to test, verify and describe the real issue.
You can create any user of any role with your admin user!

@MartinTuchyna
Copy link
Member Author

As already written, where I was able to test the issue - i have tried to do so. I was not aware of this option. When trying to create the user which role from the list in snap bellow refers to the "ROLE_CIDE_ICZ" ?

obrázok

@MartinTuchyna
Copy link
Member Author

OK, i manage to find that role - it was the last one Protection.

@MartinTuchyna
Copy link
Member Author

When I logged in with ROLE_CIDE_ICZ I could not control SMS criteria at all.
Should not be possible only for the coordinator role?

obrázok

@klimeto
Copy link
Collaborator

klimeto commented Dec 31, 2019

You can control criteria only for those CI which have SI of the protection type, because you logged as protection inspector. Now it should be possible because I fixed this issue #65

Vyrobil som SI Protection pre ES BP Dugo Selo potom nalogoval s klimeto_pr, ktory ma rolu ROLE_CIDE_ICZ a viem editorvat score aj poznamky:

image

IT works well after the fix in #65.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
out_of_scope Requirements that are not within the CIDE APP specification! testing
Projects
None yet
Development

No branches or pull requests

2 participants