Skip to content

Conversation

@max-rosin
Copy link
Contributor

@max-rosin max-rosin commented Apr 20, 2021

This PR resolves #506.

The new content is supposed to explain the general ideas behind the different top-level messages.

  • Is the information sufficient for understanding the basic idea behind these top-level messages?
  • Is important information missing?
  • Is the information correct?

Take this checklist as orientation for yourself, if this PR is ready for the Change Control Board:

  • My suggestion follows the style and contributors guidelines.
  • I have taken care about the documentation. Change concerns documentation.
  • I have done the DCO signoff.
  • My changes generate no errors when passing CI tests.
  • I have successfully implemented and tested my fix/feature locally. Documents locally rendered.
  • Appropriate reviewer(s) are assigned.

Signed-off-by: Maximilian Rosin <[email protected]>
@max-rosin max-rosin added the Documentation Everything which impacts the quality of the documentation and guidelines. label Apr 20, 2021
@max-rosin max-rosin requested a review from pmai April 20, 2021 09:04
@max-rosin max-rosin self-assigned this Apr 20, 2021
@max-rosin max-rosin added the ReadyForCCBReview Indicates that this PR is ready for a final review and merge by the CCB. label Apr 28, 2021
@stefancyliax stefancyliax added this to the V3.4.0 milestone May 1, 2021
@kmeids
Copy link

kmeids commented May 12, 2021

Output CCB 12.05.2021:

  1. WP Leads to review documentation pull requests set to "ReadyForCCB" by Max.

@clemenshabedank
Copy link
Contributor

I think TrafficCommand and TrafficUpdate is missing here.

@kmeids kmeids removed the ReadyForCCBReview Indicates that this PR is ready for a final review and merge by the CCB. label May 26, 2021
@max-rosin max-rosin added the ReadyForCCBReview Indicates that this PR is ready for a final review and merge by the CCB. label Jun 8, 2021
Copy link

@kmeids kmeids left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please check comments and suggestions

@max-rosin max-rosin merged commit 2e1665d into migration Jul 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Documentation Everything which impacts the quality of the documentation and guidelines. ReadyForCCBReview Indicates that this PR is ready for a final review and merge by the CCB.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants