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

Views #64

Merged
merged 2 commits into from
Aug 17, 2023
Merged

Views #64

merged 2 commits into from
Aug 17, 2023

Conversation

Kircheneer
Copy link
Contributor

@Kircheneer Kircheneer commented Aug 7, 2023

Adds views for the newly introduced models.

@Kircheneer Kircheneer changed the base branch from develop to feature_delices August 7, 2023 20:27
@Kircheneer Kircheneer force-pushed the lk-feature_delices branch 3 times, most recently from 237e50e to 02e78c4 Compare August 7, 2023 20:56
@Kircheneer
Copy link
Contributor Author

CI is (also) failing because of the failing CI of #59

@Kircheneer Kircheneer force-pushed the lk-feature_delices branch 2 times, most recently from 0e456f8 to 459216c Compare August 8, 2023 13:38
@@ -13,7 +13,7 @@
from design_builder.base import DesignJob
from design_builder.util import nautobot_version

logging.disable(logging.CRITICAL)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@abates does this need to stay in? It's been causing problems with one of the UI view test cases

Copy link
Contributor

Choose a reason for hiding this comment

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

I think I put this in because otherwise the tests generate a ton of logging output that isn't suppressed by the unit test framework. If you take it out do the tests run without a ton of output?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Tests are silent for me even without this 🤷🏻

@Kircheneer Kircheneer force-pushed the lk-feature_delices branch 2 times, most recently from 041b5d8 to 2b4bb07 Compare August 16, 2023 16:32
@Kircheneer Kircheneer marked this pull request as ready for review August 16, 2023 16:32
Implements views for:
- design
- design instance
- journal
- journal entry
@abates abates merged commit a23177f into feature_delices Aug 17, 2023
7 of 8 checks passed
@abates abates deleted the lk-feature_delices branch August 17, 2023 11:33
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