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.

Sign up for GitHub

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

Jump to bottom

Remove parameter limit enforcement #53

Closed
Katsute opened this issue Jan 20, 2021 · 1 comment · Fixed by #58
Closed

Remove parameter limit enforcement #53

Katsute opened this issue Jan 20, 2021 · 1 comment · Fixed by #58
Assignees
@Katsute
Labels
optimization
Projects
v1.0
Milestone
1.0.0

Comments

@Katsute
Copy link
Member

Katsute commented Jan 20, 2021

Let the server handle parameter limits and instead throw a bad parameters exception if the server throws it.
@Katsute Katsute added this to the 1.0.0 milestone Jan 20, 2021
@Katsute Katsute self-assigned this Jan 20, 2021
@Katsute Katsute added this to To Do in v1.0 via automation Jan 20, 2021
@Katsute
Copy link
Member Author

Katsute commented Jan 21, 2021

Method enforcement should also be removed, the server should handle these errors

@Katsute Katsute mentioned this issue Jan 21, 2021
Merged
Katsute added a commit that referenced this issue Jan 21, 2021
@Katsute
implement #53, fixes
8026c05
v1.0 automation moved this from To Do to Done Jan 21, 2021
Katsute added a commit that referenced this issue Jan 21, 2021
@Katsute
Move parameter enforcement to server-side (#58)
a4b393a 
* Fix #56

* implement #53, fixes
Katsute added a commit that referenced this issue Jul 22, 2022
@Katsute
Move parameter enforcement to server-side (#58)
e38bcc3 
* Fix #56

* implement #53, fixes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Katsute Katsute

Labels
optimization
Projects
No open projects
v1.0
  
Done
Milestone
1.0.0
Development

Successfully merging a pull request may close this issue.

Move parameter enforcement to server-side KatsuteDev/Mal4J
1 participant
@Katsute