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

Too many requests #51

Closed
SimonDamminga opened this issue Dec 11, 2018 · 4 comments
Closed

Too many requests #51

SimonDamminga opened this issue Dec 11, 2018 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@SimonDamminga
Copy link
Owner

SimonDamminga commented Dec 11, 2018

Describe the bug
Wanneer je veel events hebt in je tour dan krijg je een error die zecht dat je te veel requests aan het doen bent

To Reproduce
Steps to reproduce the behavior:

  1. Go to 'http://localhost:5000/event'
  2. Vul de invoervelden voor locatie in
  3. Click on 'Submit'

Expected behavior
Je mag zo veel mogelijk events toevoegen

Screenshots

Error message or code

To many requests

Application

  • Version: test 0.5.0

Desktop (please complete the following information):

  • OS: Windows 10
  • Browser: chrome
  • Browser version: 70
@SimonDamminga SimonDamminga added the bug Something isn't working label Dec 11, 2018
@JWOverschot
Copy link
Collaborator

@SimonDamminga Hoeveel zijn "veel events"

@SimonDamminga
Copy link
Owner Author

Een stuk of 15

@JWOverschot
Copy link
Collaborator

Dit is niet een probleem met de hoeveelheid van events maar dat er teveel request worden gemaakt in een korte tijd. Het is niet realistisches dat die in praktijk gebeurd.

Wat ik wel ga doen is een error message toevoeg als feedback voor de gebruiker. Voor als het toevallig wel gebeurt.

@JWOverschot JWOverschot self-assigned this Dec 14, 2018
@SimonDamminga
Copy link
Owner Author

Toch wil ik er wel voor zorgen dat deze error niet meer voorkomt, ik denk zelfs dat ik al weet hoe ik dat ga aanpakken aan de server kant. Die error message is goed maar ik wil er ook nog even naar kijken

JWOverschot pushed a commit to JWOverschot/Intro-Tour that referenced this issue Dec 14, 2018
SimonDamminga added a commit that referenced this issue Dec 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants