Skip to content

add report button - #27

Closed
shahriaarrr wants to merge 4 commits into
birlug:mainfrom
shahriaarrr:main
Closed

add report button#27
shahriaarrr wants to merge 4 commits into
birlug:mainfrom
shahriaarrr:main

Conversation

@shahriaarrr

Copy link
Copy Markdown
Contributor

fixed #26

@amiremohamadi

amiremohamadi commented Oct 3, 2025

Copy link
Copy Markdown
Member

thanks for the contribution! please fix the ci. the tests are failing now

@shahriaarrr

Copy link
Copy Markdown
Contributor Author

thanks for the contribution! please fix the ci. the tests are failing now

fixed @amiremohamadi

@amiremohamadi

Copy link
Copy Markdown
Member

did you check the scenario? it produces internal error whenever i click on the Report button.

2025_10_03_21_10_38

2025:10:03:21:12:46

@amiremohamadi

Copy link
Copy Markdown
Member

also consider that we're using the free plan, so it shouldn't reach the free plan limits. maybe d1 is a better choice? it requires an R&D

@shahriaarrr

Copy link
Copy Markdown
Contributor Author

also consider that we're using the free plan, so it shouldn't reach the free plan limits. maybe d1 is a better choice? it requires an R&D

We’ve already fixed the issue, so hitting the free plan limits shouldn’t be a problem. D1 could be a better pick, but as you said, it’ll need some R&D.

@amiremohamadi

Copy link
Copy Markdown
Member

closed as stale. feel free to open a new PR in case u wanted to continue working on it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

admin "report" button to flag spammers

2 participants