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: Display usage of T4C repositories in projects #1752

Merged
merged 2 commits into from
Oct 10, 2024
Merged

Conversation

MoritzWeber0
Copy link
Member

@MoritzWeber0 MoritzWeber0 commented Sep 4, 2024

Breaking Changes

A list of API changes is available here: #1752 (comment)
Breaking Changes are marked with a warning icon.

Description

Displays the usage of TeamForCapella repositories in projects. To avoid circular imports, I had to add two models2 modules.

@MoritzWeber0 MoritzWeber0 marked this pull request as draft September 4, 2024 16:13
Copy link

sonarcloud bot commented Sep 4, 2024

Copy link

codecov bot commented Sep 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.72%. Comparing base (08e9331) to head (07caa44).
Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1752      +/-   ##
==========================================
+ Coverage   84.69%   84.72%   +0.02%     
==========================================
  Files         203      206       +3     
  Lines        6712     6735      +23     
  Branches      465      464       -1     
==========================================
+ Hits         5685     5706      +21     
- Misses        866      867       +1     
- Partials      161      162       +1     

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

Copy link

github-actions bot commented Oct 8, 2024

A Storybook preview is available for commit f1abb04.
➡️ View Storybook
➡️ View Chromatic build

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

@MoritzWeber0 MoritzWeber0 marked this pull request as ready for review October 9, 2024 13:57

This comment has been minimized.

This comment has been minimized.

@MoritzWeber0 MoritzWeber0 changed the base branch from main to api-changelog-generation October 10, 2024 12:10

This comment has been minimized.

@MoritzWeber0 MoritzWeber0 force-pushed the api-changelog-generation branch 3 times, most recently from e13a945 to 594f33e Compare October 10, 2024 12:33
Base automatically changed from api-changelog-generation to main October 10, 2024 12:37
Copy link

github-actions bot commented Oct 10, 2024

API Changelog 4.7.2.dev22+gf1abb04a

GET /api/v1/events

  • added the required property '/items/project/id' to the response with the '200' status

GET /api/v1/projects

  • added the required property '/items/id' to the response with the '200' status

POST /api/v1/projects

  • added the required property 'id' to the response with the '200' status

GET /api/v1/projects/{project_slug}

  • added the required property 'id' to the response with the '200' status

PATCH /api/v1/projects/{project_slug}

  • added the required property 'id' to the response with the '200' status

GET /api/v1/projects/{project_slug}/events

  • added the required property 'items/items/project/id' to the response with the '200' status

GET /api/v1/projects/{project_slug}/models

  • ⚠️ removed the required property '/items/t4c_models/items/repository/instance/cdo_port' from the response with the '200' status
  • ⚠️ removed the required property '/items/t4c_models/items/repository/instance/host' from the response with the '200' status
  • ⚠️ removed the required property '/items/t4c_models/items/repository/instance/http_port' from the response with the '200' status
  • ⚠️ removed the required property '/items/t4c_models/items/repository/instance/license_server' from the response with the '200' status
  • ⚠️ removed the required property '/items/t4c_models/items/repository/instance/port' from the response with the '200' status
  • ⚠️ removed the required property '/items/t4c_models/items/repository/instance/protocol' from the response with the '200' status
  • ⚠️ removed the required property '/items/t4c_models/items/repository/instance/rest_api' from the response with the '200' status
  • ⚠️ removed the required property '/items/t4c_models/items/repository/instance/username' from the response with the '200' status
  • ⚠️ removed the required property '/items/t4c_models/items/repository/status' from the response with the '200' status

POST /api/v1/projects/{project_slug}/models

  • ⚠️ removed the required property 't4c_models/items/repository/instance/cdo_port' from the response with the '200' status
  • ⚠️ removed the required property 't4c_models/items/repository/instance/host' from the response with the '200' status
  • ⚠️ removed the required property 't4c_models/items/repository/instance/http_port' from the response with the '200' status
  • ⚠️ removed the required property 't4c_models/items/repository/instance/license_server' from the response with the '200' status
  • ⚠️ removed the required property 't4c_models/items/repository/instance/port' from the response with the '200' status
  • ⚠️ removed the required property 't4c_models/items/repository/instance/protocol' from the response with the '200' status
  • ⚠️ removed the required property 't4c_models/items/repository/instance/rest_api' from the response with the '200' status
  • ⚠️ removed the required property 't4c_models/items/repository/instance/username' from the response with the '200' status
  • ⚠️ removed the required property 't4c_models/items/repository/status' from the response with the '200' status

GET /api/v1/projects/{project_slug}/models/{model_slug}

  • ⚠️ removed the required property 't4c_models/items/repository/instance/cdo_port' from the response with the '200' status
  • ⚠️ removed the required property 't4c_models/items/repository/instance/host' from the response with the '200' status
  • ⚠️ removed the required property 't4c_models/items/repository/instance/http_port' from the response with the '200' status
  • ⚠️ removed the required property 't4c_models/items/repository/instance/license_server' from the response with the '200' status
  • ⚠️ removed the required property 't4c_models/items/repository/instance/port' from the response with the '200' status
  • ⚠️ removed the required property 't4c_models/items/repository/instance/protocol' from the response with the '200' status
  • ⚠️ removed the required property 't4c_models/items/repository/instance/rest_api' from the response with the '200' status
  • ⚠️ removed the required property 't4c_models/items/repository/instance/username' from the response with the '200' status
  • ⚠️ removed the required property 't4c_models/items/repository/status' from the response with the '200' status

PATCH /api/v1/projects/{project_slug}/models/{model_slug}

  • ⚠️ removed the required property 't4c_models/items/repository/instance/cdo_port' from the response with the '200' status
  • ⚠️ removed the required property 't4c_models/items/repository/instance/host' from the response with the '200' status
  • ⚠️ removed the required property 't4c_models/items/repository/instance/http_port' from the response with the '200' status
  • ⚠️ removed the required property 't4c_models/items/repository/instance/license_server' from the response with the '200' status
  • ⚠️ removed the required property 't4c_models/items/repository/instance/port' from the response with the '200' status
  • ⚠️ removed the required property 't4c_models/items/repository/instance/protocol' from the response with the '200' status
  • ⚠️ removed the required property 't4c_models/items/repository/instance/rest_api' from the response with the '200' status
  • ⚠️ removed the required property 't4c_models/items/repository/instance/username' from the response with the '200' status
  • ⚠️ removed the required property 't4c_models/items/repository/status' from the response with the '200' status

GET /api/v1/projects/{project_slug}/models/{model_slug}/backups/pipelines

  • ⚠️ removed the required property '/items/t4c_model/instance_name' from the response with the '200' status
  • ⚠️ removed the required property '/items/t4c_model/project_name' from the response with the '200' status
  • ⚠️ removed the required property '/items/t4c_model/repository_name' from the response with the '200' status
  • added the required property '/items/t4c_model/id' to the response with the '200' status
  • added the required property '/items/t4c_model/name' to the response with the '200' status
  • added the required property '/items/t4c_model/repository' to the response with the '200' status

POST /api/v1/projects/{project_slug}/models/{model_slug}/backups/pipelines

  • ⚠️ removed the required property 't4c_model/instance_name' from the response with the '200' status
  • ⚠️ removed the required property 't4c_model/project_name' from the response with the '200' status
  • ⚠️ removed the required property 't4c_model/repository_name' from the response with the '200' status
  • added the required property 't4c_model/id' to the response with the '200' status
  • added the required property 't4c_model/name' to the response with the '200' status
  • added the required property 't4c_model/repository' to the response with the '200' status

GET /api/v1/projects/{project_slug}/models/{model_slug}/backups/pipelines/{pipeline_id}

  • ⚠️ removed the required property 't4c_model/instance_name' from the response with the '200' status
  • ⚠️ removed the required property 't4c_model/project_name' from the response with the '200' status
  • ⚠️ removed the required property 't4c_model/repository_name' from the response with the '200' status
  • added the required property 't4c_model/id' to the response with the '200' status
  • added the required property 't4c_model/name' to the response with the '200' status
  • added the required property 't4c_model/repository' to the response with the '200' status

GET /api/v1/projects/{project_slug}/models/{model_slug}/modelsources/t4c

  • ⚠️ removed the required property '/items/repository/instance/cdo_port' from the response with the '200' status
  • ⚠️ removed the required property '/items/repository/instance/host' from the response with the '200' status
  • ⚠️ removed the required property '/items/repository/instance/http_port' from the response with the '200' status
  • ⚠️ removed the required property '/items/repository/instance/license_server' from the response with the '200' status
  • ⚠️ removed the required property '/items/repository/instance/port' from the response with the '200' status
  • ⚠️ removed the required property '/items/repository/instance/protocol' from the response with the '200' status
  • ⚠️ removed the required property '/items/repository/instance/rest_api' from the response with the '200' status
  • ⚠️ removed the required property '/items/repository/instance/username' from the response with the '200' status
  • ⚠️ removed the required property '/items/repository/status' from the response with the '200' status

POST /api/v1/projects/{project_slug}/models/{model_slug}/modelsources/t4c

  • ⚠️ removed the required property 'repository/instance/cdo_port' from the response with the '200' status
  • ⚠️ removed the required property 'repository/instance/host' from the response with the '200' status
  • ⚠️ removed the required property 'repository/instance/http_port' from the response with the '200' status
  • ⚠️ removed the required property 'repository/instance/license_server' from the response with the '200' status
  • ⚠️ removed the required property 'repository/instance/port' from the response with the '200' status
  • ⚠️ removed the required property 'repository/instance/protocol' from the response with the '200' status
  • ⚠️ removed the required property 'repository/instance/rest_api' from the response with the '200' status
  • ⚠️ removed the required property 'repository/instance/username' from the response with the '200' status
  • ⚠️ removed the required property 'repository/status' from the response with the '200' status

GET /api/v1/projects/{project_slug}/models/{model_slug}/modelsources/t4c/{t4c_model_id}

  • ⚠️ removed the required property 'repository/instance/cdo_port' from the response with the '200' status
  • ⚠️ removed the required property 'repository/instance/host' from the response with the '200' status
  • ⚠️ removed the required property 'repository/instance/http_port' from the response with the '200' status
  • ⚠️ removed the required property 'repository/instance/license_server' from the response with the '200' status
  • ⚠️ removed the required property 'repository/instance/port' from the response with the '200' status
  • ⚠️ removed the required property 'repository/instance/protocol' from the response with the '200' status
  • ⚠️ removed the required property 'repository/instance/rest_api' from the response with the '200' status
  • ⚠️ removed the required property 'repository/instance/username' from the response with the '200' status
  • ⚠️ removed the required property 'repository/status' from the response with the '200' status

PATCH /api/v1/projects/{project_slug}/models/{model_slug}/modelsources/t4c/{t4c_model_id}

  • ⚠️ removed the required property 'repository/instance/cdo_port' from the response with the '200' status
  • ⚠️ removed the required property 'repository/instance/host' from the response with the '200' status
  • ⚠️ removed the required property 'repository/instance/http_port' from the response with the '200' status
  • ⚠️ removed the required property 'repository/instance/license_server' from the response with the '200' status
  • ⚠️ removed the required property 'repository/instance/port' from the response with the '200' status
  • ⚠️ removed the required property 'repository/instance/protocol' from the response with the '200' status
  • ⚠️ removed the required property 'repository/instance/rest_api' from the response with the '200' status
  • ⚠️ removed the required property 'repository/instance/username' from the response with the '200' status
  • ⚠️ removed the required property 'repository/status' from the response with the '200' status

GET /api/v1/settings/modelsources/t4c/instances/{t4c_instance_id}/repositories

  • ⚠️ removed the required property 'payload/items/instance/cdo_port' from the response with the '200' status
  • ⚠️ removed the required property 'payload/items/instance/host' from the response with the '200' status
  • ⚠️ removed the required property 'payload/items/instance/http_port' from the response with the '200' status
  • ⚠️ removed the required property 'payload/items/instance/license_server' from the response with the '200' status
  • ⚠️ removed the required property 'payload/items/instance/port' from the response with the '200' status
  • ⚠️ removed the required property 'payload/items/instance/protocol' from the response with the '200' status
  • ⚠️ removed the required property 'payload/items/instance/rest_api' from the response with the '200' status
  • ⚠️ removed the required property 'payload/items/instance/username' from the response with the '200' status
  • the 'payload/items/name' response's property pattern '^[-a-zA-Z0-9_]+$' was removed for the status '200'
  • added the required property 'payload/items/integrations' to the response with the '200' status

GET /api/v1/settings/modelsources/t4c/license-servers

  • added the required property '/items/instances/items/is_archived' to the response with the '200' status
  • added the required property '/items/instances/items/version' to the response with the '200' status

POST /api/v1/settings/modelsources/t4c/license-servers

  • added the required property 'instances/items/is_archived' to the response with the '200' status
  • added the required property 'instances/items/version' to the response with the '200' status

GET /api/v1/settings/modelsources/t4c/license-servers/{t4c_license_server_id}

  • added the required property 'instances/items/is_archived' to the response with the '200' status
  • added the required property 'instances/items/version' to the response with the '200' status

PATCH /api/v1/settings/modelsources/t4c/license-servers/{t4c_license_server_id}

  • added the required property 'instances/items/is_archived' to the response with the '200' status
  • added the required property 'instances/items/version' to the response with the '200' status

GET /api/v1/users/{user_id}/common-projects

  • added the required property '/items/id' to the response with the '200' status

GET /api/v1/users/{user_id}/events

  • added the required property '/items/project/id' to the response with the '200' status

This comment has been minimized.

This comment has been minimized.

The numbers beginning with '#' were translated to issue / PR links,
breaking the API Changelog.
Copy link

sonarcloud bot commented Oct 10, 2024

Copy link

This report was generated by comparing f1abb04 with 08e9331.
If you would like to check difference, please check here.

change detected

ArtifactName: reg

item count
pass 320
change 20
new 2
delete 0
📝 Report

Differences

Pipeline Components_Pipeline Deletion Dialog_As Admin_desktop.png

actual Actual
expected Expected
difference Difference

Pipeline Components_Pipeline Deletion Dialog_As Admin_mobile.png

actual Actual
expected Expected
difference Difference

Pipeline Components_Pipeline Deletion Dialog_As Project Admin_desktop.png

actual Actual
expected Expected
difference Difference

Pipeline Components_Pipeline Deletion Dialog_As Project Admin_mobile.png

actual Actual
expected Expected
difference Difference

Pipeline Components_Pipeline Deletion Dialog_Request Sent_desktop.png

actual Actual
expected Expected
difference Difference

Pipeline Components_Pipeline Deletion Dialog_Request Sent_mobile.png

actual Actual
expected Expected
difference Difference

Pipeline Components_Trigger Pipeline_Pipeline Overview_desktop.png

actual Actual
expected Expected
difference Difference

Pipeline Components_Trigger Pipeline_Pipeline Overview_mobile.png

actual Actual
expected Expected
difference Difference

Settings Components_Modelsources_T4C_Repositories_Add Repositories_desktop.png

actual Actual
expected Expected
difference Difference

Settings Components_Modelsources_T4C_Repositories_Add Repositories_mobile.png

actual Actual
expected Expected
difference Difference

Settings Components_Modelsources_T4C_Repositories_Repositories_desktop.png

actual Actual
expected Expected
difference Difference

Settings Components_Modelsources_T4C_Repositories_Repositories_mobile.png

actual Actual
expected Expected
difference Difference

Settings Components_Modelsources_T4C_Server Instance_Add Instance_desktop.png

actual Actual
expected Expected
difference Difference

Settings Components_Modelsources_T4C_Server Instance_Add Instance_mobile.png

actual Actual
expected Expected
difference Difference

Settings Components_Modelsources_T4C_Server Instance_Archived Instance_desktop.png

actual Actual
expected Expected
difference Difference

Settings Components_Modelsources_T4C_Server Instance_Archived Instance_mobile.png

actual Actual
expected Expected
difference Difference

Settings Components_Modelsources_T4C_Server Instance_Edit Existing Instance_desktop.png

actual Actual
expected Expected
difference Difference

Settings Components_Modelsources_T4C_Server Instance_Edit Existing Instance_mobile.png

actual Actual
expected Expected
difference Difference

Settings Components_Modelsources_T4C_Server Instance_Existing Instance_desktop.png

actual Actual
expected Expected
difference Difference

Settings Components_Modelsources_T4C_Server Instance_Existing Instance_mobile.png

actual Actual
expected Expected
difference Difference

New Items

Settings Components_Modelsources_T4C_Repositories_Repositories With Integrations_desktop.png

NewItem

Settings Components_Modelsources_T4C_Repositories_Repositories With Integrations_mobile.png

NewItem

@MoritzWeber0 MoritzWeber0 merged commit a28c242 into main Oct 10, 2024
31 checks passed
@MoritzWeber0 MoritzWeber0 deleted the usage-of-t4c branch October 10, 2024 13:22
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.

1 participant