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

Request on landholding form : override the limitation is possible ! #101

Closed
MaelREBOUX opened this issue Feb 26, 2021 · 3 comments
Closed

Comments

@MaelREBOUX
Copy link
Member

I can ask more than 5 demands there for I reached the limitation of 5 / week.

https://portail-test.sig.rennesmetropole.fr/cadastrapp/services/checkRequestLimitation?cni=maeltest&type=P3

The backend is clearly saying : "0 request left for this person"

{"requestAvailable":0,"user":{"commune":"","firstName":"mael","lastName":"reboux","adress":"","cni":"maeltest","codePostal":"","userId":1,"mail":"","type":"P3"}}

But the form let me ask some new demands. This can't be possible.

Could you reproduce @landryb @jusabatier ?

image

@jusabatier
Copy link
Collaborator

Reproduced here, but the print buttons still gayed and not clickable.

I agree that a red warning would be better.

@offtherailz
Copy link
Collaborator

It's ok to show the same message in the request object list, like when you are adding too many entries?
image.png

offtherailz added a commit to offtherailz/mapstore2-cadastrapp that referenced this issue Mar 3, 2021
@offtherailz
Copy link
Collaborator

I created this PR for this change, please let me know if it is ok for you, I'll merge.
#110 (comment)

@tdipisa tdipisa added this to the VA MS2 milestone Mar 24, 2021
tdipisa pushed a commit that referenced this issue Mar 30, 2021
#101 show max request message when the limit is 0
@tdipisa tdipisa closed this as completed Mar 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants