-
Notifications
You must be signed in to change notification settings - Fork 2
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 Customer Support Ticket ID field to Posts #166
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This commit adds a new property called "CustomerSupportTicketId" to the Post model. This property is used to store the Customer Support ticket ID associated with a post. It can take a positive integer value up to 9 digits or be set as null if no ticket is associated. The purpose of this property is to allow WG staff to track and manage player reports. Implements database requirements for #165.
This commit adds a new field, SupportTicketStatus, to the PlayerPostDTO class in the WowsKarma.Common project. The SupportTicketStatus field represents the status of the Customer Support ticket associated with the post when applicable. Additionally, mappings are configured in the Conversions utility class to map the CustomerSupportTicketId property of Post to TicketId property of SupportTicketStatus in PlayerPostDTO. This allows for seamless conversion between Post and PlayerPostDTO objects while preserving the relevant information about Customer Support tickets.
- Added a new property `supportTicketStatus` to the `PlayerPostEditorDto` interface and implemented it in the `PostEditorComponent` - Created a new component `CsTicketIdHelpComponent` to display information about Customer Support tickets - Added a new HTML template file for the `CsTicketIdHelpComponent` - Updated the HTML template of the `PostEditorComponent` to include a CS ticket ID field when certain conditions are met - Updated the CSS styles of the `SeedTokenChangeComponent` (removed) - Updated the TypeScript code of the `SeedTokenChangeComponent` (removed) - Updated the HTML template of the `PostComponent` to display CS ticket related icons and tooltips Implements frontend requirements for #165.
The code changes in this commit fix an issue where the support ticket ID was not being attributed correctly to a post. The commit adds logic to retrieve the ticket ID from the form controls and assigns it to the post's supportTicketStatus property. This ensures that the post is associated with the correct support ticket.
This commit adds a new field "CS Ticket ID" to the Discord embed in the PostWebhookService class. The field displays the support ticket ID if it exists in the post's SupportTicketStatus. This change enhances the information displayed in the Discord webhook message.
SakuraIsayeki
added
API
Related to API development
enhancement
New feature or request
Web
Related to Web App development
labels
Sep 1, 2024
- Updated the HTML template of the post component to improve the display of support tickets. - Added a condition in the HTML template to show the support ticket ID only if the user is the owner or privileged. - Changed the tooltip text from "Reported to CS" to "Reported to Customer Support". - Modified the TypeScript file of the post component to include a computed property `isOwnerOrPrivileged` that checks if the user is the owner or has privileged roles. This commit improves how support tickets are displayed and provides more accurate information for users.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
API
Related to API development
enhancement
New feature or request
Web
Related to Web App development
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Special thanks to @Ahskance for this proposal.
Closes #165.