Skip to content

[backend] Security platform module + FF (#10452)#11045

Merged
SouadHadjiat merged 24 commits intorelease/currentfrom
issue/10452-chunk1
May 26, 2025
Merged

[backend] Security platform module + FF (#10452)#11045
SouadHadjiat merged 24 commits intorelease/currentfrom
issue/10452-chunk1

Conversation

@CelineSebe
Copy link
Member

@CelineSebe CelineSebe commented May 19, 2025

Proposed changes

Related issues

Checklist

  • I consider the submitted work as finished
  • I tested the code for its functionality
  • I wrote test cases for the relevant uses case (coverage and e2e)
  • I added/update the relevant documentation (either on github or on notion)
  • Where necessary I refactored code to improve the overall quality

Further comments

@github-actions github-actions bot added the filigran team use to identify PR from the Filigran team label May 19, 2025
@CelineSebe CelineSebe changed the base branch from master to release/current May 19, 2025 14:04
@CelineSebe CelineSebe changed the title [frontend] Security platform module [backend] Security platform module May 20, 2025
@CelineSebe CelineSebe changed the title [backend] Security platform module [backend] Security platform module (#10452) May 20, 2025
@CelineSebe CelineSebe force-pushed the issue/10452-chunk1 branch from c7ab728 to fdfd357 Compare May 20, 2025 09:45
@CelineSebe CelineSebe self-assigned this May 20, 2025
@CelineSebe CelineSebe changed the title [backend] Security platform module (#10452) [backend] Security platform module + FF (#10452) May 20, 2025
@CelineSebe CelineSebe force-pushed the issue/10452-chunk1 branch from 75ca7bf to 07d31af Compare May 20, 2025 13:02
@CelineSebe CelineSebe added the multi-repository For contribution that requires PR in several repository label May 22, 2025
@CelineSebe CelineSebe force-pushed the issue/10452-chunk1 branch 2 times, most recently from 015ccca to 91e2c02 Compare May 22, 2025 14:52
@@ -0,0 +1,29 @@
import { executionContext, SYSTEM_USER } from '../utils/access';
Copy link
Member

Choose a reason for hiding this comment

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

after rebase release/current, check the migration timestamp

@codecov
Copy link

codecov bot commented May 26, 2025

Codecov Report

Attention: Patch coverage is 96.94656% with 4 lines in your changes missing coverage. Please review.

Project coverage is 65.70%. Comparing base (b9d0fc2) to head (9f5e598).
Report is 4 commits behind head on release/current.

Files with missing lines Patch % Lines
...ules/securityPlatform/securityPlatform-resolver.ts 90.90% 2 Missing ⚠️
...l/src/modules/securityPlatform/securityPlatform.ts 96.36% 2 Missing ⚠️
Additional details and impacted files
@@                 Coverage Diff                 @@
##           release/current   #11045      +/-   ##
===================================================
+ Coverage            65.40%   65.70%   +0.29%     
===================================================
  Files                  698      704       +6     
  Lines                68829    69140     +311     
  Branches              7594     7641      +47     
===================================================
+ Hits                 45015    45425     +410     
+ Misses               23814    23715      -99     

☔ 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.

@SouadHadjiat SouadHadjiat marked this pull request as ready for review May 26, 2025 12:22
@SouadHadjiat SouadHadjiat merged commit 81fcb20 into release/current May 26, 2025
6 checks passed
@SouadHadjiat SouadHadjiat deleted the issue/10452-chunk1 branch May 26, 2025 12:26
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 multi-repository For contribution that requires PR in several repository

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants