-
Notifications
You must be signed in to change notification settings - Fork 309
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
Implement MAC settings profile CRUD API #7396
Conversation
5c16798
to
93769f2
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.
Great stuff!
Should we mark this API as experimental until we complete the rest of the ToDo items this is tested with end devices? |
Yeah good call, let's do that. |
@KrishnaIyer Can you please help me out? What should I do to mark it as experimental? Is it just the CHANGELOG or more? |
|
f4ef7a5
to
b371c74
Compare
Summary
References: #7380
This PR implements the MAC settings profile CRUD API. This is the first step (1 out of 6) of the proposed implementation plan.
Changes
Testing
Steps
Pre-requisite: Create an application
Tests:
Results
Request:
Response:
Request:
Response:
Request:
Response:
Request:
Response:
Request:
Response:
Regressions
...
Notes for Reviewers
...
Checklist
README.md
for the chosen target branch.CHANGELOG.md
.CONTRIBUTING.md
, there are no fixup commits left.