Skip to content
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

Stopping point linter and viewset updates #471

Merged
merged 4 commits into from Apr 28, 2023

Conversation

itdependsnetworks
Copy link
Contributor

No description provided.

Copy link
Contributor

@jeffkala jeffkala left a comment

Choose a reason for hiding this comment

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

Will do another review once reaady.

nautobot_golden_config/api/serializers.py Show resolved Hide resolved
@jeffkala jeffkala self-requested a review April 17, 2023 17:25
Copy link
Contributor

@jeffkala jeffkala left a comment

Choose a reason for hiding this comment

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

Nice, pretty cool stat as far as code reduction with UI Viewsets. 55 additions 294 deletions!!!

@jeffkala
Copy link
Contributor

During testing noticing that on Rules, Features, Replacements, Removals, the "add" views have duplicated Notes section.
Screen Shot 2023-04-25 at 3 43 34 PM

@jeffkala
Copy link
Contributor

During testing noticing that on Rules, Features, Replacements, Removals, the "add" views have duplicated Notes section. Screen Shot 2023-04-25 at 3 43 34 PM

looks like this must have been fixed in 1.5.x, tested again in 1.5.4 and it doesnt' show this any longer.

@jeffkala
Copy link
Contributor

Been playing with this for awhile now, seems great and working as expected.

One thing that I'm not seeing is buttons on thecompliance feature detail view.
Screen Shot 2023-04-27 at 7 46 58 AM

…ancefeature_retrieve.html

Co-authored-by: Jeff Kala <48843785+jeffkala@users.noreply.github.com>
@jeffkala
Copy link
Contributor

Now that the buttons issue is fixed everything else with this PR continues to function as expected. Caveat is Notes being duplicated in 1.4, I'd say we may want to just bump min support up to 1.5.3.

@itdependsnetworks
Copy link
Contributor Author

Agreed. Will do that before next release

@itdependsnetworks itdependsnetworks merged commit adf575b into nautobot:develop Apr 28, 2023
16 checks passed
jmpettit pushed a commit to jmpettit/nautobot-app-golden-config that referenced this pull request Jan 30, 2024
* Stopping point linter and viewset updates

* More updates

* pylint updates

* Update nautobot_golden_config/templates/nautobot_golden_config/compliancefeature_retrieve.html

Co-authored-by: Jeff Kala <48843785+jeffkala@users.noreply.github.com>

---------

Co-authored-by: Jeff Kala <48843785+jeffkala@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants