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

bug 1122 fix editing notification sendTo attribute #1327

Merged
merged 1 commit into from
Jul 30, 2024

Conversation

ds-lcapellino
Copy link
Contributor

@ds-lcapellino ds-lcapellino commented Jul 29, 2024

Description

Pre-review checks

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

resolves #1122

Copy link
Contributor

github-actions bot commented Jul 29, 2024

✅ No Dependency Check findings were found

Copy link
Contributor

github-actions bot commented Jul 29, 2024

Integration Test Results

348 tests   347 ✅  1m 47s ⏱️
 41 suites    1 💤
 41 files      0 ❌

Results for commit 417b436.

♻️ This comment has been updated with latest results.

Copy link
Contributor

github-actions bot commented Jul 29, 2024

Unit Test Results

209 tests   209 ✅  18s ⏱️
 58 suites    0 💤
 58 files      0 ❌

Results for commit 417b436.

♻️ This comment has been updated with latest results.

@ds-mwesener ds-mwesener marked this pull request as ready for review July 30, 2024 06:22
Copy link
Contributor

@ds-mwesener ds-mwesener left a comment

Choose a reason for hiding this comment

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

See comments.

@ds-lcapellino ds-lcapellino force-pushed the bug/1122-fix-notification-editing branch 2 times, most recently from 5ef6f49 to 417b436 Compare July 30, 2024 06:49
@ds-lcapellino ds-lcapellino force-pushed the bug/1122-fix-notification-editing branch from 417b436 to e60a433 Compare July 30, 2024 06:54
Copy link
Contributor

@ds-mwesener ds-mwesener 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-mwesener ds-mwesener merged commit b4f5bac into main Jul 30, 2024
19 checks passed
@ds-mwesener ds-mwesener deleted the bug/1122-fix-notification-editing branch July 30, 2024 06:56
Copy link

sonarcloud bot commented Jul 30, 2024

Quality Gate Passed Quality Gate passed for 'eclipse-tractusx_traceability-foss-frontend'

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link

sonarcloud bot commented Jul 30, 2024

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.

🐞[BUG] Editing "sendTo" BPN changes "createdBy" BPN
2 participants