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

Add user reports #158

Merged
merged 3 commits into from
Oct 13, 2024
Merged

Add user reports #158

merged 3 commits into from
Oct 13, 2024

Conversation

Zentro
Copy link
Member

@Zentro Zentro commented Sep 1, 2024

This adds 3 commands, 1 for the user to use:
!report <uid> <message>
!reports
!viewreport <rid>

How to test: Report another test account, see if the report is properly added. View the report and ensure the report message, IP, and reporter are all properly displayed.

@CuriousMike56
Copy link
Member

Submitting report:
RoR_2024-09-15_13-41-00
WindowsTerminal_2024-09-15_13-41-09

No output from the !reports command on either client.

@ohlidalp
Copy link
Member

ohlidalp commented Oct 4, 2024

I'm not at home this weekend, will review next week.

@ohlidalp
Copy link
Member

I noticed the !report command doesn't display the usage hint - you must type it !report (followed by space) to get the usage hint, which I think is confusing considering this is for regular players rather than mods who read docs.

... funny enough, all commands are like this. For no good reason as far as I'm concerned. But let's tackle that in another PR.

Copy link
Member

@ohlidalp ohlidalp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested on Windows 10 x64, works well.

@ohlidalp ohlidalp merged commit de6b796 into RigsOfRods:master Oct 13, 2024
2 checks passed
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.

3 participants