-
Notifications
You must be signed in to change notification settings - Fork 10
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
feat(network): add board change #862
Conversation
Maybe a good idea to mention that all of this builds on AEGEE/network#2 |
48eae83
to
beb2b79
Compare
9762ad5
to
4db0ef7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In general looks good already, nice work!
24e95cd
to
64afc7f
Compare
@serge1peshcoff this is working now so after the final tests of Network Director/NetCom we can merge this and publish the first thing of the network module |
This is of course merged with AEGEE/network#2 and AEGEE/mailer#65 |
@WikiRik do you maybe know what the status of this PR is? |
We need to rebase and retest but I think this is ready to be merged apart from that |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can merge this in, without the manage_network:boards or view:board permissions nobody should see this
# [1.34.0](1.33.37...1.34.0) (2023-09-26) ### Features * **network:** add board change ([#862](#862)) ([32e44a9](32e44a9))
🎉 This PR is included in version 1.34.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Implementes MA-140 (hopefully)