daily icon indicating copy to clipboard operation
daily copied to clipboard

🐛 BUG: Not showing error message when a user reaches RATE_LIMITED of daily new post

Open henriqueweiand opened this issue 11 months ago • 3 comments

What went wrong? 🤔

I'm a new user of Daily News, and since I have posted a few articles on other platforms, I wanted to add them to Daily News as well. After creating a squad -> https://app.daily.dev/squads/nestjsninja, I started posting. I posted two articles as links, and then when I tried to add more, my post wasn't created like the others were, and I didn't receive an error message. I tried one more time with the console open, and I noticed that I reached the daily limit, which I wasn't aware of. The bug report that I'd like to open is related to the user feedback that is not being shown. Thanks!

Image

The response message {"response":{"data":{"submitExternalLink":null},"errors":[{"message":"Take a break. You already posted enough","locations":[{"line":9,"column":5}],"path":["submitExternalLink"],"extensions":{"code":"RATE_LIMITED"}}],"status":200,"headers":{}},"request":{"query":"\n mutation SubmitExternalLink(\n $sourceId: ID!\n $url: String!\n $title: String\n $image: String\n $commentary: String\n ) {\n submitExternalLink(\n url: $url\n title: $title\n image: $image\n sourceId: $sourceId\n commentary: $commentary\n ) {\n _\n }\n }\n","variables":{"url":"https://medium.com/nestjs-ninja/how-to-integrate-multiple-payment-gateways-in-nestjs-with-stripe-example-978be76af17f","title":"How to Integrate Multiple Payment Gateways in NestJS (With Stripe Example)","image":"https://miro.medium.com/v2/resize:fit:1024/1*nwaktCJcfEcgZthHmFlJjA.jpeg","sourceId":"f7c2d86d-8003-45f6-b039-cbcc85f9caff","commentary":"https://medium.com/nestjs-ninja/how-to-integrate-multiple-payment-gateways-in-nestjs-with-stripe-example-978be76af17f"}}}

Expected Behavior

Receive the error message when the RATE_LIMITED happens

Steps to Reproduce Issue

1. Create multiple post on the squad
2. With the console open get the error messsage

Solution Proposed

Update the interface to get the error and show it to the user

Screenshots

No response

Environment

No response

Browsers

Chrome

OS

Mac

Version of daily.dev

No response

Additional Context


Code of Conduct

  • [x] I follow the conditions of this project.

henriqueweiand avatar Jan 23 '25 17:01 henriqueweiand

Hi @henriqueweiand,

Thanks for the report, we will fix it 🙏

idoshamun avatar Jan 26 '25 07:01 idoshamun

Hello, me and @heenriquepinheiro would like to work on this issue. Could it be assigned to us?

BeatrizAlmeida03 avatar Mar 17 '25 15:03 BeatrizAlmeida03

Hi @BeatrizAlmeida03,

You can go ahead and use our toast component to show a message

idoshamun avatar Mar 18 '25 07:03 idoshamun