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

more context commands #535

Merged
merged 1 commit into from
Nov 25, 2023
Merged

more context commands #535

merged 1 commit into from
Nov 25, 2023

Conversation

Ixrec
Copy link
Contributor

@Ixrec Ixrec commented Nov 25, 2023

No description provided.

@Ixrec Ixrec force-pushed the more-context-commands branch from 74fdd30 to eda9796 Compare November 25, 2023 13:49
@Ixrec Ixrec merged commit 880643a into Manechat:mane Nov 25, 2023
2 checks passed
@@ -270,6 +284,10 @@
}
}

// hardcoding Manechat specific role ids
private ulong banishedRoleId = 368961099925553153;

Check notice

Code scanning / CodeQL

Missed 'readonly' opportunity Note

Field 'banishedRoleId' can be 'readonly'.
@@ -270,6 +284,10 @@
}
}

// hardcoding Manechat specific role ids
private ulong banishedRoleId = 368961099925553153;
private ulong memberRoleId = 552450130633031700;

Check notice

Code scanning / CodeQL

Missed 'readonly' opportunity Note

Field 'memberRoleId' can be 'readonly'.
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.

1 participant