Thanks to visit codestin.com
Credit goes to github.com

Skip to content

allow for empty policy...to(...) call #353

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
Closed
catmando opened this issue Feb 18, 2021 · 1 comment
Closed

allow for empty policy...to(...) call #353

catmando opened this issue Feb 18, 2021 · 1 comment
Labels
enhancement New feature or request ready-to-release Internal Use Only: Has been fixed, specs passing and pushed to edge branch

Comments

@catmando
Copy link
Contributor

In a instance broadcast regulation ( policy.send...to(list of channels) ) there is a check designed to insure that the .to method is called, since it is sometimes forgotten.

The problem the check fails even if .to is called but the list of channels is empty.

Probably the easist thing is to remove the check, and document the need for the .to method.
@catmando catmando added the bug Something isn't working label Feb 18, 2021
@catmando catmando reopened this Mar 15, 2021
@catmando catmando added enhancement New feature or request needs doc Everything is working, but documentation is needed. ready-to-release Internal Use Only: Has been fixed, specs passing and pushed to edge branch and removed bug Something isn't working labels Mar 15, 2021
@catmando
Copy link
Contributor Author

catmando commented Mar 15, 2021 •
edited
Loading

This has been fixed properly. System now checks if to is forgotten, but ignores if to provides an empty set.

@catmando catmando removed the needs doc Everything is working, but documentation is needed. label Mar 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request ready-to-release Internal Use Only: Has been fixed, specs passing and pushed to edge branch
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@catmando