Skip to content

Some Feature Updates#314

Merged
zhiyan114 merged 18 commits into
mainfrom
feat
Feb 16, 2024
Merged

Some Feature Updates#314
zhiyan114 merged 18 commits into
mainfrom
feat

Conversation

@zhiyan114

@zhiyan114 zhiyan114 commented Dec 31, 2023

Copy link
Copy Markdown
Collaborator

Resolves #230

  • @camfsd are you able to give a try and see if the feat works as intended.

Test Note:

  • ticketTypes should now have staffRoles as array (though it can be undefined or empty array if not in use). Confirm with the example config if needed.
  • rolesWhoHaveAccessToTheTickets is now a "global access" role

Resolves #248

  • @RaiRaii I would still recommend you to use add commands instead, but in-case you want to mass add users (which does have a couple optimizations), you can use the new massadd command. Also do test it to confirm if it's working or not.

Test Note:

  • It's a standard slash command: massadd
    • It takes in a string argument (which removes the build-in user validation by discord).
    • User ID will be used as input
    • To add multiple users, use the comma , as the separator

Resolves #323
Unable to reproduce the said issue, perhaps the fix is already in this branch?


Unrelated

  • Added clearDM command to allow users to clean-up history DMs from the bot

@zhiyan114 zhiyan114 self-assigned this Dec 31, 2023
@zhiyan114 zhiyan114 added this to the 3.3.1 milestone Dec 31, 2023
@zhiyan114

Copy link
Copy Markdown
Collaborator Author

@camfsd @RaiRaii Let me know how the software is going before I start the merging process.

@zhiyan114 zhiyan114 marked this pull request as ready for review February 15, 2024 20:46
@zhiyan114 zhiyan114 requested a review from Sayrix as a code owner February 15, 2024 20:46
@Sayrix

Sayrix commented Feb 15, 2024

Copy link
Copy Markdown
Owner

Can you delete the package-lock.json just to see if after that the license compliance passes

@zhiyan114

zhiyan114 commented Feb 15, 2024

Copy link
Copy Markdown
Collaborator Author

In that case, then it has to be one of the sub-dependencies that's causing the issue.

@Sayrix Sayrix left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

lgtm, just some copyrights who are still in 2023 (massadd.ts for example)

@Sayrix

Sayrix commented Feb 16, 2024

Copy link
Copy Markdown
Owner

Lmk if you want to push the version btw

@zhiyan114

Copy link
Copy Markdown
Collaborator Author

Alr, we'll prob do it after we organize with the new translation feature.

@zhiyan114 zhiyan114 merged commit a2712af into main Feb 16, 2024
@zhiyan114 zhiyan114 deleted the feat branch February 16, 2024 17:39
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.

Closed Ticket Access Issue: Users Unable to Open New Tickets After Closure /Add role per-category staff roles

3 participants