Skip to content

Conversation

gabriel-peze
Copy link

Proposed changes

  • Add payload details on inject update modal

Related issues

@gabriel-peze gabriel-peze self-assigned this Sep 8, 2025
@gabriel-peze gabriel-peze changed the title Issue/3769 Issue/3769 Add payload details to inject update Sep 8, 2025
@gabriel-peze gabriel-peze added the filigran team use to identify PR from the Filigran team label Sep 9, 2025
@MarineLeM
Copy link
Contributor

MarineLeM commented Sep 9, 2025

Not all the injects are linked to a payload, so I guess in this case we should remove the "payload details" tab ?
image

@savacano28
Copy link
Contributor

savacano28 commented Sep 9, 2025

Observations:

  1. I'm having some errors when I try update an atomic testing (I dont have any info in inject detail):
image 2. Is it possible make the call just when we are on the tab? image image 3. Filter out injects without contract (email, challenge, nuclei... http..) image

@savacano28
Copy link
Contributor

Tested OK!

@gabriel-peze gabriel-peze merged commit dc335ff into release/current Sep 10, 2025
4 checks passed
@gabriel-peze gabriel-peze deleted the issue/3769 branch September 10, 2025 07:07
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.

3 participants