-
Notifications
You must be signed in to change notification settings - Fork 401
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
S7szl #287
S7szl #287
Conversation
LGTM ... only thing I didn't quite understand is why a subscription field has multiple fields inside. |
I try to resolve the conflicts |
ok might be easier to rebase this... Merge was bit hard an I aborted it due to time constraints... |
Hello, |
no I tried the rebase locally and noticed that it might be easier to rebase the changes onto current develop. Due to timeconstraints I couldn't finish it in time. I try to finish it this weekend. |
@glcj I propose to close this PR and you reopen a new one based on a branch in this repo (feature/s7_subscription. There you need to apply your changes manually and we try then to get them merged ASAP. |
Add the possibility of: