-
Notifications
You must be signed in to change notification settings - Fork 399
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
fix(driver): reject multiple calls #484
Conversation
style: remove empty line
Stale pull request message |
This could also be fixed by appending an empty handler to the list of middlewares when registering the action in the driver so that express stops handling the request. |
@Elassyo, I took over and I'm trying to first categorize and prioritize what can/need to be fixed first (that's why you see some bot activity lately). |
@jotamorais Use bot to shut down prs & issues rather than deal with them, it's just a cover up, not a solution. I want every user to see these processes, and watch out for managers' inaction, and please use this opensource project very very carefully |
@unix, the idea to add the bot and to flag stale and close stale issues and PRs was to trigger action (and as you can see, triggered your action too). We had over 150 open issues and 18 old PRs (some open since 2017). No one is trying to cover up or not deal with the issues (we didn't delete the issue - it's closed with a stale flag) and we will get back to it at some point. I certainly appreciate your PR and any contribution, but there's no need to shout this kind of statements out in the wind. On the contrary, we should be calling for more ideas and contributions on how to put the project back on track. What are your suggestions? |
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Reject action duplicate calls caused by unsatisfactory design.
About issue: #220 .
fixed #220 .