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

Implement core tickets system #20

Merged
merged 72 commits into from
Jan 5, 2024
Merged

Implement core tickets system #20

merged 72 commits into from
Jan 5, 2024

Conversation

No767
Copy link
Member

@No767 No767 commented Dec 11, 2023

This PR addresses #8, #10 and #19 . The features being implemented are the following:

  • Ticket creation and lifecycle
  • User webhook replying back in the ticket forum
  • Staff notifications/logging for ticket creation and closure
  • Ticket closure command
  • Basic info and active commands for users
  • Replying back as an admin back to the user in thread

And there are changes to views as well. In addition, there are some remaining tasks that needs to be addressed before this PR can get reviewed and merged, notability:

  • Replying back to the user from an admin
  • File attachment support for tickets
  • Implement basic info and active commands
  • Temp cache and selection process of tags
  • Allow for custom user titles (w/ defaults?)
  • Cleaner implementation of the ?close command
  • Remove the notification system as this may stress folks more
  • Implement proper logging w/ embeds (such as when a ticket is created and closed) - But how would this information be used to plan rational decisions in the future?

And some bugs that needs to be ironed out:

  • Ticket Creation View keeps on editing an unknown message
  • Caches not being properly invalidated

@No767 No767 marked this pull request as ready for review December 17, 2023 08:31
@No767 No767 self-assigned this Dec 19, 2023
@No767 No767 requested a review from jamievlin December 28, 2023 09:21
No767 and others added 24 commits January 2, 2024 02:12
Copy link

sonarcloud bot commented Jan 5, 2024

Quality Gate Passed Quality Gate passed

The SonarCloud Quality Gate passed, but some issues were introduced.

2 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@No767 No767 merged commit 92f2972 into main Jan 5, 2024
9 checks passed
@No767 No767 deleted the noelle/tickets branch January 6, 2024 00:07
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.

2 participants