This repository has been archived by the owner on Apr 26, 2024. It is now read-only.
Dev/ux migration models fix #503
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Dev/ux migration
* fix ux-2 schema breaking models, fix transform * remove URL(Allow/Block)Parcel from overloaded methods
* Push UX2 - Create config group and feature profiles * Simplify push_ux2_config * UX2Config rework * Refactor code for template definition normalization This commit refactors the code responsible for normalizing template definitions by implementing improved type annotations, better function organization, and enhanced error handling. Key changes include: 1. Refactoring the to_snake_case function to use the str.replace method for kebab-case to snake_case conversion. 2. Improving the cast_value_to_global function to handle different types of input values, including lists and IP addresses, and utilizing IPv4Address and IPv6Address from the ipaddress module. 3. Enhancing the transform_dict function to handle nested dictionaries and lists, casting leaf values to global types recursively. 4. Updating the template_definition_normalization function to utilize the refactored transform_dict function for key transformation and value normalization. These changes improve code readability, maintainability, and robustness, providing a more efficient and reliable solution for template definition normalization. * Add tests for normalization and Literal casting --------- Co-authored-by: Kuba <[email protected]> Co-authored-by: sbasan <[email protected]> Co-authored-by: Jakub Krajewski <[email protected]>
…dpoints Dev/featureprofile system endpoints
update models for sdwandemo items
Added Feature Profile System Models for UX2
sbasan
requested review from
kagrski,
JimOverholt,
igosoft and
bboot2
as code owners
5 tasks
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Pull Request summary:
[Describe briefly what the PR is about]
Description of changes:
[Add more in depth analysis of what changed, provide logs, examples of usage]
Checklist: