Skip to content

Conversation

@savacano28
Copy link
Contributor

@savacano28 savacano28 commented Nov 21, 2025

Proposed changes

  • Add constraint unicity for asset name ,platform type (for security platform)
  • Update ui to includ platform in options/resutls..
image

Testing Instructions

Inject/Atomic Testing

  1. If you change to branche master and you add more injectExpectation Result from securities platforms with the same name
    after running this pr you will be able to see just one result for an unique security platform.

  1. Add expectationResult for a security platform
  2. Add again, check that prevoius security platform is anymore present in the options
  3. Delete expectation Result, check you can add again a new result from this sec platform

CRUD Security Platform

  1. Create securityPlatform
  2. Create new security platfor with the same type and name ->Error
  3. Create a new securityplatform
  4. Try update last security platform as teh same name of first security platform -> Error

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
  • I added/update the relevant documentation (either on github or on notion)
  • Where necessary I refactored code to improve the overall quality
  • For bug fix -> I implemented a test that covers the bug

Further comments

If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

@savacano28 savacano28 self-assigned this Nov 21, 2025
@savacano28 savacano28 added the filigran team use to identify PR from the Filigran team label Nov 21, 2025
@savacano28 savacano28 changed the title [backend/frontend] fix(security-platforms): deduplication security platforms (#2346) [backend/frontend] fix: deduplication security platforms (#2346) Nov 21, 2025
@savacano28 savacano28 force-pushed the issue/2346 branch 4 times, most recently from c27d5ba to 213f4be Compare January 9, 2026 15:42
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.

Security Platforms should be de-duplicated on the name to avoid mistakes

2 participants