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.

Sign up for GitHub

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

Jump to bottom

Removal of investigation/alerts for parts as planned #674

Merged
merged 26 commits into from
Feb 19, 2024
Merged

Removal of investigation/alerts for parts as planned #674

merged 26 commits into from
Feb 19, 2024

Conversation

ds-mwesener
Copy link
Contributor

Description

Added

  • Endpoint (assets/import/report/{importJobId}) for retrieving import report

Removed

  • Investigations/Alerts for assets_as_planned parts

Pre-review checks

Please ensure to do as many of the following checks as possible, before asking for committer review:

ds-lcapellino and others added 26 commits February 13, 2024 10:33
@ds-lcapellino
feature: 416 add job information when importing assets
bdd59c0
@ds-lcapellino
feature: 416 add job information when importing assets
109b9e1
@ds-lcapellino
feature: 416 add get endpoint to retrieve import job report
8b7576e
@ds-lcapellino
feature: 416 add integration test, update arc42 and swagger
462b49c
@ds-lcapellino
Merge branch 'main' into feature/416-get-report-data-from-import
6ef58dd 
# Conflicts:
#	CHANGELOG.md
@ds-lcapellino
feature: 416 add not found integration test scenario, refactor
c7bfdcf
@ds-lcapellino
feature: 416 fix untit tests
d259e36
@ds-lcapellino
feature: 416 add scenario documentation
5dd0608
@ds-lcapellino
feature: 416 add scenario documentation
bf1384b
@ds-ext-sceronik
Merge pull request #987 from eclipse-tractusx/main
019ca20 
Merge pull request #657 from catenax-ng/main
@ds-lcapellino
feature: 416 cancel job in case of errors
f8912de
@ds-lcapellino
Merge branch 'main' into feature/416-get-report-data-from-import
55b137b
@ds-lcapellino
Merge pull request #982 from catenax-ng/feature/416-get-report-data-f…
756723c 
…rom-import

Feature/416 get report data from import
@ds-mmaul
chore(security): XXX fix medium security findings from veracode
7845dc7
@ds-mmaul
Merge pull request #988 from catenax-ng/chore/XXX-fix-security-issues
ea21418 
chore(security): XXX fix medium security findings from veracode (hotfix)
@ds-mmaul @github-actions
Update Dependencies Backend Action
5b2063f
@ds-mwesener
Merge pull request #986 from catenax-ng/action/dependencies
5779270 
Update Dependencies Backend
@ds-mwesener
chore(security): 516 removed investigation / alert from asPlanned ent…
0a4bbc5 
…ities
@ds-mwesener
chore(security): 516 removed investigation / alert from asPlanned ent…
31b653c 
…ities
@ds-mwesener
chore(security): 516 removed investigation / alert from asPlanned ent…
a661120 
…ities
@ds-mwesener
chore(security): 516 cleared database tables for notifications on as …
de28d29 
…planned.
@ds-mwesener
chore(security): 516 cleared database tables for notifications on as …
2c592d5 
…planned.
@ds-mwesener
chore(security): 516 removed unused import.
429c898
@ds-mwesener
chore(security): 516 removed unused import.
af8af6c
@ds-mwesener
chore(security): 516 removed unused import.
e36446d
@ds-mwesener
Merge pull request #990 from catenax-ng/chore/516-removal-asplanned-i…
367f15c 
…nvestigations

chore(security): 516 removed investigation / alert from asPlanned ent…
ds-mmaul
ds-mmaul approved these changes Feb 19, 2024
View reviewed changes
Copy link
Contributor

@ds-mmaul ds-mmaul left a comment

Choose a reason for hiding this comment

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

LGTM

@ds-mmaul ds-mmaul merged commit d6f1618 into eclipse-tractusx:main Feb 19, 2024
13 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ds-mmaul ds-mmaul ds-mmaul approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@ds-mwesener @ds-mmaul @ds-lcapellino @ds-ext-sceronik