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

Feature Request #8

Open
patricius opened this issue Jun 9, 2018 · 3 comments
Open

Feature Request #8

patricius opened this issue Jun 9, 2018 · 3 comments

Comments

@patricius
Copy link

https://www.nationstates.net/cgi-bin/api.cgi?q=happenings;region=europe;filter=move;limit=50%2522

Hi. Sorry to bother you, but I have a request.

Could this be added as a filter?

For instance, I'm using your region, it would continuously message people who leave it. so it would filter off the departed from take the nation making the action and the add it to recipients list.

region:Europe, +action:departed would be the syntax

@ifly6
Copy link
Owner

ifly6 commented Dec 14, 2020

Really really late. It isn't addable because the way the program is structured requires there to be an enumerated list of recipients at start. The recruitment daemon works on the back end by simply re-running the enumeration every 180 seconds. If I were to redesign the program, I'd pick a different structure.

I'm rewriting the base library not to take an enumerated list of recipients but rather to process them in a queue. Already wrote some code which would do what you requested; it isn't entirely ready yet. Due to the change though, I'm re-opening this Issue.

@ifly6 ifly6 closed this as completed Dec 14, 2020
@ifly6 ifly6 reopened this Dec 15, 2020
@ifly6
Copy link
Owner

ifly6 commented Dec 16, 2020

Current implementation uses API calls for regional inhabitants rather than monitoring the happenings call. This is mostly because I set up the framework expecting that new possibilities are provided faster than telegrams can be sent. See CommMovementMonitor.

@ifly6 ifly6 added this to the Dynamic recipient generation milestone Dec 16, 2020
@ifly6
Copy link
Owner

ifly6 commented Apr 7, 2024

This should now be present in the current pre-release code base for version 13 as flag _movement:out_of;europe.

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

No branches or pull requests

2 participants