Skip to content

Review and Refactor: dev and dev_title Feature Flags in /mention-each Command #307

@shobhan-sundar-goutam

Description

@shobhan-sundar-goutam

Issue Description

The /mention-each slash command currently uses two separate feature flags:

  • dev
  • dev_title

During execution, these flags manage how users with specific roles are mentioned and displayed.

However, having two separate flags for a single command flow seems unnecessary and may lead to confusion or inconsistent behavior. The command has been stable for a while now, and it's a good time to revisit this logic.

Expected Behavior

Please conduct a thorough review of the flag usage in this command by referring the doc and the code. Based on the review, either:

  • Cleanly remove the flags if they are no longer needed, consolidating the logic.
  • Refactor or simplify the behavior if only one flag is necessary.
  • Or, clearly document and justify why both flags are still required if they are to be kept.

Severity/Priority

  • Critical
  • High
  • Medium
  • Low

Checklist

  • I have read and followed the project's code of conduct.
  • I have searched for similar issues before creating this one.
  • I have provided all the necessary information to understand and reproduce the issue.
  • I am willing to contribute to the resolution of this issue.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions