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

Switch DeviceGroupForm in seeddb to new uncrispy way #3122

Merged
merged 1 commit into from
Oct 10, 2024

Conversation

johannaengland
Copy link
Contributor

Changes again most changes done in #2994 and instead uses our newest way of uncrispifying forms.

Urls:
http://localhost/seeddb/netboxgroup/add/
http://localhost/seeddb/netboxgroup/edit/<id>

@johannaengland johannaengland self-assigned this Oct 9, 2024
@johannaengland johannaengland changed the title Refactor/switch seeddb to new uncrispy way Switch DeviceGroupForm in seeddb to new uncrispy way Oct 9, 2024
@johannaengland johannaengland force-pushed the refactor/switch-seeddb-to-new-uncrispy-way branch from 55c3cf2 to 83a634b Compare October 9, 2024 10:59
Copy link

github-actions bot commented Oct 9, 2024

🦙 MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ PYTHON black 992 0 11.0s
✅ PYTHON ruff 987 0 0.1s

See detailed report in MegaLinter reports

MegaLinter is graciously provided by OX Security

Copy link

github-actions bot commented Oct 9, 2024

Test results

    9 files      9 suites   8m 22s ⏱️
2 135 tests 2 135 ✅ 0 💤 0 ❌
4 009 runs  4 009 ✅ 0 💤 0 ❌

Results for commit 8339f1e.

♻️ This comment has been updated with latest results.

Copy link

codecov bot commented Oct 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 60.44%. Comparing base (6a7754b) to head (8339f1e).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3122   +/-   ##
=======================================
  Coverage   60.44%   60.44%           
=======================================
  Files         605      605           
  Lines       43761    43762    +1     
  Branches       48       48           
=======================================
+ Hits        26451    26452    +1     
  Misses      17298    17298           
  Partials       12       12           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

python/nav/web/templates/seeddb/edit.html Outdated Show resolved Hide resolved
@johannaengland johannaengland requested a review from stveit October 10, 2024 12:50
This removes our old idea of adding templates and uses the new way that reimplemented crispy fields
@johannaengland johannaengland force-pushed the refactor/switch-seeddb-to-new-uncrispy-way branch from cce0dee to 8339f1e Compare October 10, 2024 13:33
Copy link

sonarcloud bot commented Oct 10, 2024

@johannaengland johannaengland merged commit 8ed1822 into master Oct 10, 2024
13 of 14 checks passed
@johannaengland johannaengland deleted the refactor/switch-seeddb-to-new-uncrispy-way branch October 10, 2024 14:01
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.

2 participants