Skip to content

fix: remove SECURITY_ALERTS_API_ENABLED flag #14221

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

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

vinistevam
Copy link
Contributor

@vinistevam vinistevam commented Mar 25, 2025

Description

This PR aims to remove the SECURITY_ALERTS_API_ENABLED flag.

Related issues

Fixes: https://github.com/MetaMask/mobile-planning/issues/2041

Manual testing steps

  1. Go to test dapp
  2. In the section `` test all transactions and signatures and every request should be validated by Security Alert API properly.

Screenshots/Recordings

test_secuirty-alert-api.webm

Before

After

Pre-merge author checklist

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

@vinistevam vinistevam added the team-confirmations Push issues to confirmations team label Mar 25, 2025
Copy link
Contributor

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@vinistevam vinistevam marked this pull request as ready for review March 25, 2025 13:10
@vinistevam vinistevam requested review from a team as code owners March 25, 2025 13:10
@vinistevam vinistevam added the Run Smoke E2E Triggers smoke e2e on Bitrise label Mar 25, 2025
Copy link
Contributor

github-actions bot commented Mar 25, 2025

https://bitrise.io/ Bitrise

❌❌❌ pr_smoke_e2e_pipeline failed on Bitrise! ❌❌❌

Commit hash: 1d036b9
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/a5189b44-f7c7-44cd-ac37-2839eed6bc40

Note

  • You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

Tip

  • Check the documentation if you have any doubts on how to understand the failure on bitrise

@vinistevam vinistevam added Run Smoke E2E Triggers smoke e2e on Bitrise and removed Run Smoke E2E Triggers smoke e2e on Bitrise labels Mar 28, 2025
Copy link
Contributor

github-actions bot commented Mar 28, 2025

https://bitrise.io/ Bitrise

✅✅✅ pr_smoke_e2e_pipeline passed on Bitrise! ✅✅✅

Commit hash: 8bef7a7
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/50af2bed-a271-42de-83f8-b5dcd9a55d9c

Note

  • You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 65.72%. Comparing base (b539406) to head (8bef7a7).
Report is 5 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #14221      +/-   ##
==========================================
- Coverage   65.72%   65.72%   -0.01%     
==========================================
  Files        2228     2228              
  Lines       47433    47433              
  Branches     6682     6682              
==========================================
- Hits        31176    31175       -1     
  Misses      14263    14263              
- Partials     1994     1995       +1     

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

@vinistevam vinistevam added Run Smoke E2E Triggers smoke e2e on Bitrise and removed Run Smoke E2E Triggers smoke e2e on Bitrise labels Apr 21, 2025
Copy link
Contributor

github-actions bot commented Apr 21, 2025

https://bitrise.io/ Bitrise

❌❌❌ pr_smoke_e2e_pipeline failed on Bitrise! ❌❌❌

Commit hash: 5d45465
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/e338ed8b-9fe4-4392-9dda-39ec35825eb8

Note

  • You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

Tip

  • Check the documentation if you have any doubts on how to understand the failure on bitrise

@jpuri
Copy link
Contributor

jpuri commented Apr 21, 2025

We may also need to delete PPOM bundling code from mobile.

@vinistevam vinistevam added Run Smoke E2E Triggers smoke e2e on Bitrise and removed Run Smoke E2E Triggers smoke e2e on Bitrise labels Apr 22, 2025
Copy link
Contributor

github-actions bot commented Apr 22, 2025

https://bitrise.io/ Bitrise

✅✅✅ pr_smoke_e2e_pipeline passed on Bitrise! ✅✅✅

Commit hash: 364bd81
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/d4f27cc1-8a35-40af-aac7-dcd83a183437

Note

  • You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

@vinistevam vinistevam added the no-changelog Indicates no external facing user changes, therefore no changelog documentation needed label Apr 23, 2025
@vinistevam vinistevam added the No QA Needed Apply this label when your PR does not need any QA effort. label Apr 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No QA Needed Apply this label when your PR does not need any QA effort. no-changelog Indicates no external facing user changes, therefore no changelog documentation needed Run Smoke E2E Triggers smoke e2e on Bitrise team-confirmations Push issues to confirmations team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants