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

Adds in-game bug reports without needing a GitHub account #6392

Merged
merged 106 commits into from
Jun 19, 2024
Merged

Adds in-game bug reports without needing a GitHub account #6392

merged 106 commits into from
Jun 19, 2024

Conversation

vero5123
Copy link
Contributor

@vero5123 vero5123 commented Jun 7, 2024

About the pull request

Adds in-game bug reports without needing a GitHub account

Explain why it's good for the game

Essentially, the idea is to allow for the creation of bug reports in-game without needing to own a Github account, this can help to incentivize issue reporting and be beneficial overall.

TODO:

  • Implement the Github API token to allow for anonymous bug reports
  • Make in-game bug reports admin approval only to prevent abuse and to make sure only quality bug reports are approved.

Notable changes from Goonstation's bug report system is that I migrated it to TypeScript and it now requires admin approval.

Changelog

🆑
add: Adds the ability for users to make bug reports in-game
ui: New bug report system ui
/:cl:

@github-actions github-actions bot added UI deletes nanoui/html Feature Feature coder badge labels Jun 7, 2024
@vero5123 vero5123 marked this pull request as draft June 7, 2024 02:00
@harryob harryob added the Do Not Merge If you merge this PR, I will annihilate you label Jun 7, 2024
@harryob
Copy link
Member

harryob commented Jun 7, 2024

this requires the original devs of this feature on goon to license this agpl-ey, goon's cc-by-nc-sa is not compatible

@pali6
Copy link

pali6 commented Jun 7, 2024

Heya, original author of the goonstation feature here. You have my permission to port this under whichever license you use. Enjoy.

@vero5123
Copy link
Contributor Author

vero5123 commented Jun 7, 2024

Heya, original author of the goonstation feature here. You have my permission to port this under whichever license you use. Enjoy.

Thank you!

@harryob harryob removed the Do Not Merge If you merge this PR, I will annihilate you label Jun 8, 2024
This was referenced Jun 16, 2024
@harryob harryob self-assigned this Jun 16, 2024
cm13-github added a commit that referenced this pull request Jun 16, 2024
cm13-github added a commit that referenced this pull request Jun 16, 2024
cm13-github added a commit that referenced this pull request Jun 16, 2024
cm13-github added a commit that referenced this pull request Jun 16, 2024
cm13-github added a commit that referenced this pull request Jun 16, 2024
@Harrysno Harrysno mentioned this pull request Jun 17, 2024
3 tasks
cm13-github added a commit that referenced this pull request Jun 17, 2024
@harryob harryob added this pull request to the merge queue Jun 19, 2024
Merged via the queue into cmss13-devs:master with commit b9a0286 Jun 19, 2024
27 checks passed
cm13-github added a commit that referenced this pull request Jun 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Feature coder badge Testmerge Candidate we'll test this while you're asleep and the server has 10 players UI deletes nanoui/html
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants