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

[Other]: Clarification Needed: Understanding the Difference Between CheckAdmin and CheckIMAdmin Functions #2115

Closed
cubxxw opened this issue Mar 18, 2024 · 1 comment
Labels
other Indicates an issue or PR lacks a `triage/foo` label and requires one. support Categorizes issue or PR as a support question.

Comments

@cubxxw
Copy link
Contributor

cubxxw commented Mar 18, 2024

What would you like to share?

Description:
The repository contains two functions named CheckAdmin and CheckIMAdmin, which appear to serve similar purposes of checking user permissions. However, it's not entirely clear from the code or existing documentation what the distinction between these two functions is. This issue aims to request clarification or documentation explaining the difference between these functions and when each should be used.

Possible Actions:

  1. Provide a detailed explanation in the code comments or README file about the specific use cases for CheckAdmin and CheckIMAdmin functions.
  2. Update existing documentation to include a section that clearly outlines the differences and usage scenarios for these functions.
  3. If there are no significant differences between these functions, consider consolidating them into a single function with configurable parameters to handle both cases.

Additional information

Expected Outcome:
Developers should have a clear understanding of when to use CheckAdmin versus CheckIMAdmin functions, thereby improving code clarity and maintainability.

@cubxxw cubxxw added other Indicates an issue or PR lacks a `triage/foo` label and requires one. support Categorizes issue or PR as a support question. labels Mar 18, 2024
@kubbot
Copy link
Contributor

kubbot commented May 27, 2024

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
other Indicates an issue or PR lacks a `triage/foo` label and requires one. support Categorizes issue or PR as a support question.
Projects
None yet
Development

No branches or pull requests

3 participants