-
Notifications
You must be signed in to change notification settings - Fork 4
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
feat: Add possibilities to archive t4c instances #969
Conversation
bf51bbc
to
91a4b2a
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #969 +/- ##
==========================================
+ Coverage 71.06% 71.15% +0.08%
==========================================
Files 155 156 +1
Lines 4869 4891 +22
Branches 522 524 +2
==========================================
+ Hits 3460 3480 +20
- Misses 1312 1313 +1
- Partials 97 98 +1
☔ View full report in Codecov by Sentry. |
91a4b2a
to
9579da7
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.
looks already good to me
...rc/app/settings/modelsources/t4c-settings/edit-t4c-instance/edit-t4c-instance.component.html
Outdated
Show resolved
Hide resolved
.../src/app/settings/modelsources/t4c-settings/edit-t4c-instance/edit-t4c-instance.component.ts
Outdated
Show resolved
Hide resolved
Use |
3922000
to
28f1647
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.
Looks good and works perfectly. Just three minor improvement suggestions.
backend/capellacollab/alembic/versions/f7bf9456cfc9_add_archive_flag.py
Outdated
Show resolved
Hide resolved
...rc/app/settings/modelsources/t4c-settings/edit-t4c-instance/edit-t4c-instance.component.html
Outdated
Show resolved
Hide resolved
28f1647
to
d282337
Compare
428c1f9
to
5d25ebb
Compare
eb6c2f0
to
b73f50c
Compare
b73f50c
to
fbda95b
Compare
This sets the port when running mkdocs to 8081, so as not to interfere with a running backend on 8080.
fbda95b
to
4c4bfe2
Compare
This PR adds an archive flag to the t4c instances. When set the t4c instances should be grayed out in the drop down menu (i.e., no longer be selectable) and one should no longer be able to edit the instance. However, existing linked models should still work as expected.
Resolves #970