Skip to content

[backend] fix(import): import paylaod with remediation (#5244)#5376

Merged
gabriel-peze merged 7 commits intomasterfrom
issue/5244
Apr 17, 2026
Merged

[backend] fix(import): import paylaod with remediation (#5244)#5376
gabriel-peze merged 7 commits intomasterfrom
issue/5244

Conversation

@MarineLeM
Copy link
Copy Markdown
Contributor

Proposed changes

  • Import payload with remediation

Related issues

@MarineLeM MarineLeM self-assigned this Apr 7, 2026
@MarineLeM MarineLeM added the filigran team use to identify PR from the Filigran team label Apr 7, 2026
@MarineLeM MarineLeM changed the title [backend] fix(import): import paylaod with remediation [backend] fix(import): import paylaod with remediation (#5244) Apr 7, 2026
@gabriel-peze gabriel-peze self-assigned this Apr 15, 2026
@gabriel-peze gabriel-peze marked this pull request as ready for review April 15, 2026 13:55
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 15, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 58.61%. Comparing base (b0a4c54) to head (9b0e367).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #5376   +/-   ##
=========================================
  Coverage     58.61%   58.61%           
- Complexity     4863     4866    +3     
=========================================
  Files          1030     1030           
  Lines         30427    30436    +9     
  Branches       2254     2254           
=========================================
+ Hits          17834    17841    +7     
- Misses        11543    11545    +2     
  Partials       1050     1050           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@EmilieFo17
Copy link
Copy Markdown
Contributor

🤖 [PRWatch] — No test files detected in this PR.

This bug-fix PR (linked to #5244) does not appear to include any test coverage for the changes made. Please consider adding tests before merging.

@gabriel-peze gabriel-peze force-pushed the issue/5244 branch 2 times, most recently from 7bb257b to e4c4dbb Compare April 16, 2026 14:40
@gabriel-peze
Copy link
Copy Markdown
Contributor

Hey @RomuDeuxfois

Every feedback have been tackle, if you have time you can recheck 😊

Comment thread openaev-api/src/test/java/io/openaev/api/payload/PayloadApiImporterTest.java Outdated
@gabriel-peze gabriel-peze merged commit 38ab15b into master Apr 17, 2026
14 checks passed
@gabriel-peze gabriel-peze deleted the issue/5244 branch April 17, 2026 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

filigran team use to identify PR from the Filigran team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unable to export payloads containing remediation

4 participants