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

feat: Add possibilities to archive t4c instances #969

Merged
merged 4 commits into from
Oct 13, 2023

Conversation

dominik003
Copy link
Contributor

@dominik003 dominik003 commented Aug 28, 2023

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

@dominik003 dominik003 marked this pull request as draft August 28, 2023 09:20
@dominik003 dominik003 changed the title Add possibilities to archive t4c instances feat: Add possibilities to archive t4c instances Sep 4, 2023
@dominik003 dominik003 force-pushed the feat-archive-t4c-server branch 3 times, most recently from bf51bbc to 91a4b2a Compare September 11, 2023 13:16
@codecov
Copy link

codecov bot commented Sep 11, 2023

Codecov Report

Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (e2bd86b) 71.06% compared to head (4c4bfe2) 71.15%.

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     
Files Coverage Δ
backend/capellacollab/__main__.py 73.61% <100.00%> (+0.75%) ⬆️
...ellacollab/settings/modelsources/t4c/exceptions.py 100.00% <100.00%> (ø)
.../capellacollab/settings/modelsources/t4c/models.py 95.60% <100.00%> (+0.20%) ⬆️
.../capellacollab/settings/modelsources/t4c/routes.py 100.00% <100.00%> (ø)
...lab/projects/toolmodels/modelsources/t4c/routes.py 67.56% <0.00%> (ø)
...llacollab/settings/modelsources/t4c/injectables.py 71.42% <66.66%> (-6.35%) ⬇️

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

@dominik003 dominik003 force-pushed the feat-archive-t4c-server branch from 91a4b2a to 9579da7 Compare September 18, 2023 09:40
@dominik003 dominik003 marked this pull request as ready for review September 18, 2023 14:07
Copy link
Contributor

@Paula-Kli Paula-Kli left a 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

@dominik003
Copy link
Contributor Author

dominik003 commented Sep 26, 2023

Use archive as archive icon

@dominik003 dominik003 force-pushed the feat-archive-t4c-server branch 2 times, most recently from 3922000 to 28f1647 Compare September 28, 2023 07:02
Copy link
Member

@MoritzWeber0 MoritzWeber0 left a 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.

@Paula-Kli Paula-Kli force-pushed the feat-archive-t4c-server branch from 28f1647 to d282337 Compare October 6, 2023 15:05
@dominik003 dominik003 force-pushed the feat-archive-t4c-server branch 2 times, most recently from 428c1f9 to 5d25ebb Compare October 9, 2023 10:36
@dominik003 dominik003 force-pushed the feat-archive-t4c-server branch 3 times, most recently from eb6c2f0 to b73f50c Compare October 9, 2023 16:26
@dominik003 dominik003 force-pushed the feat-archive-t4c-server branch from b73f50c to fbda95b Compare October 9, 2023 17:01
This sets the port when running mkdocs to 8081,
so as not to interfere with a running backend on 8080.
@dominik003 dominik003 force-pushed the feat-archive-t4c-server branch from fbda95b to 4c4bfe2 Compare October 9, 2023 17:13
@MoritzWeber0 MoritzWeber0 dismissed Paula-Kli’s stale review October 13, 2023 11:46

Comments are resolved

@MoritzWeber0 MoritzWeber0 merged commit 984f451 into main Oct 13, 2023
23 checks passed
@MoritzWeber0 MoritzWeber0 deleted the feat-archive-t4c-server branch October 13, 2023 11:46
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.

Archive TeamForCapella servers
3 participants