Skip to content
This repository was archived by the owner on Feb 3, 2025. It is now read-only.

feat(Structures): Add PermissionOverwrite #35

Open
wants to merge 7 commits into
base: beta
Choose a base branch
from
Open

feat(Structures): Add PermissionOverwrite #35

wants to merge 7 commits into from

Conversation

SiroTM
Copy link
Collaborator

@SiroTM SiroTM commented Jun 9, 2021

This PR makes the permission_overwrites property optional, sets it's type to PermissionOverwrite and changes the index.ts file.

bsian03
bsian03 suggested changes Jun 9, 2021
View reviewed changes
@Khaaz Khaaz changed the title refactor(Structures): Channel feat(Structures): Add PermissionOverwrite Jun 12, 2021
@Khaaz Khaaz added repo: structures semver: feat t: addition t: documentation Improvements or additions to documentation labels Jun 12, 2021
Khaaz
Khaaz approved these changes Jun 12, 2021
View reviewed changes
Copy link
Collaborator

@Khaaz Khaaz 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, I'll let bsian double check ts

bsian03
bsian03 suggested changes Jun 12, 2021
View reviewed changes
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@bsian03 bsian03 bsian03 requested changes

@Khaaz Khaaz Khaaz approved these changes

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
repo: structures semver: feat t: addition t: documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@SiroTM @bsian03 @Khaaz