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.

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

Is it allowed to switch interval time by sending new subscription request? #52

Closed
Tracked by #78
otterdahl opened this issue Jun 1, 2020 · 3 comments
Closed
Tracked by #78

Is it allowed to switch interval time by sending new subscription request? #52

otterdahl opened this issue Jun 1, 2020 · 3 comments
Labels
clarify
Milestone
3.2

Comments

@otterdahl
Copy link
Contributor

Is it allowed to switch interval time by sending new subscription request while the subscription is already active?
@emiltin
Copy link
Contributor

emiltin commented Jun 1, 2020

You could argue that it's not much different from unsubscribing and subscribing with a new interval. I would allow it.

@emiltin emiltin added the clarify label Sep 7, 2021
@emiltin emiltin added this to the 3.2 milestone Sep 16, 2021
otterdahl added a commit that referenced this issue Nov 8, 2021
@otterdahl
Clarify subscriptions
41e86e1 
* Use bullet lists to make it easier to read
* It is allowed to switch interval time by sending a new subscription
  request

Discussion in #52
@otterdahl
Copy link
Contributor Author

A suggestion of clarification added to the draft

@otterdahl otterdahl mentioned this issue Nov 8, 2021
Merged
18 tasks
otterdahl added a commit that referenced this issue Nov 8, 2021
@otterdahl
Clarify subscriptions (2)
86b76dc 
Fix typo

Discussion in #52
@otterdahl
Copy link
Contributor Author

I'm closing this. Feel free to open this issue again if you have any feedback on the draft

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
clarify
Projects
None yet
Milestone
3.2
Development

No branches or pull requests
2 participants
@emiltin @otterdahl