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

Fixed ticket creation errors. #572

Merged
merged 1 commit into from
Jan 5, 2025
Merged

Conversation

DominicTheD3v
Copy link
Contributor

@DominicTheD3v DominicTheD3v commented Jan 3, 2025

Versioning information

  • This includes major changes (breaking changes)
  • This includes minor changes (minimal usage changes, minor new features)
  • This includes patches (bug fixes)
  • This does not change functionality at all (code refactoring, comments)

Is this related to an issue?

No.

Changes made

I have fixed the "Invalid User or Role" error occurring once a ticket is opened. The Channel Permissions Override for the Ticket Channel was referencing the @everyone role object instead of the associated role id. This caused a TypeError upon ticket usage.

Confirmations

  • I have updated related documentation (if necessary)
  • My changes use consistent code style
  • My changes have been tested and confirmed to work

@RooRay RooRay assigned RooRay and eartharoid and unassigned RooRay Jan 4, 2025
Copy link
Contributor

@RooRay RooRay left a comment

Choose a reason for hiding this comment

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

looks good to me, no obvious issues

@RooRay RooRay requested a review from eartharoid January 4, 2025 02:26
@eartharoid eartharoid merged commit 5ccf58c into discord-tickets:main Jan 5, 2025
2 checks passed
@eartharoid
Copy link
Member

@all-contributors please add @DominicTheD3v for bug and code

Copy link
Contributor

@eartharoid

I've put up a pull request to add @DominicTheD3v! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Complete
Development

Successfully merging this pull request may close these issues.

3 participants