8000 ns_ajoin ADDALL Issue #363 - Added a basic function for ajoin addall by jnewing · Pull Request #379 · anope/anope · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

ns_ajoin ADDALL Issue #363 - Added a basic function for ajoin addall #379

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: 2.0
Choose a base branch
from
Open

ns_ajoin ADDALL Issue #363 - Added a basic function for ajoin addall #379

wants to merge 2 commits into from

Conversation

jnewing
Copy link
@jnewing jnewing commented Mar 10, 2024 •
edited ACF0
Loading

Summary

Adds "ADDALL" to to the ns_ajoin module. Running this command gets a list of channels the user is currently in and attempts to add them to users ajoin list if the following conditions are met.

  1. The total number of channels the user is on + the current number existing in their list may not exceed the ajoinmax parameter. (User will be informed)
  2. The channel can not have a key set. Channels without a key will be added, user will be informed of channels that have a key set and to manually add.

Rationale

This was a enhancement request #363 - I liked the idea and wanted to see it, so I did it.

Testing Environment

I have tested this pull request on:

  • Debian GNU/Linux 12 (bookworm)
  • gcc version 12.2.0-14

@SadieCat SadieCat added the enhancement New feature or request label Mar 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jnewing @SadieCat
0