Skip to content

fix: add deprecated tag back to callBackgroundMethod#25216

Merged
DDDDDanica merged 3 commits intodevelopfrom
fix/deprecated-tag
Jun 11, 2024
Merged

fix: add deprecated tag back to callBackgroundMethod#25216
DDDDDanica merged 3 commits intodevelopfrom
fix/deprecated-tag

Conversation

@DDDDDanica
Copy link
Copy Markdown
Contributor

@DDDDDanica DDDDDanica commented Jun 11, 2024

Description

Removed wrongly in #21410, this PR aims to add deprecated tag back to callBackgroundMethod for migration purpose.

Open in GitHub Codespaces

Related issues

Fixes:

Manual testing steps

  1. Go to this page...

Screenshots/Recordings

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.

@DDDDDanica DDDDDanica added the team-extension-platform Extension Platform team label Jun 11, 2024
@DDDDDanica DDDDDanica self-assigned this Jun 11, 2024
@DDDDDanica DDDDDanica requested a review from a team as a code owner June 11, 2024 12:37
@github-actions
Copy link
Copy Markdown
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.

@hjetpoluru hjetpoluru self-requested a review June 11, 2024 12:39
hjetpoluru
hjetpoluru previously approved these changes Jun 11, 2024
@DDDDDanica DDDDDanica dismissed stale reviews from hjetpoluru and pedronfigueiredo via 1151da3 June 11, 2024 13:02
@DDDDDanica DDDDDanica force-pushed the fix/deprecated-tag branch from 3492df6 to 1151da3 Compare June 11, 2024 13:02
@codecov
Copy link
Copy Markdown

codecov bot commented Jun 11, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 65.59%. Comparing base (bc6acbe) to head (5405fb3).
Report is 4 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop   #25216   +/-   ##
========================================
  Coverage    65.59%   65.59%           
========================================
  Files         1362     1362           
  Lines        54167    54167           
  Branches     14167    14167           
========================================
  Hits         35527    35527           
  Misses       18640    18640           

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

@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [1151da3]
Page Load Metrics (45 ± 4 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint6012276136
domContentLoaded9131011
load39814594
domInteractive9131011
Bundle size diffs
  • background: 0 Bytes (0.00%)
  • ui: 0 Bytes (0.00%)
  • common: 0 Bytes (0.00%)

@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [5405fb3]
Page Load Metrics (137 ± 170 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint6510588126
domContentLoaded9261242
load431679137354170
domInteractive9261242
Bundle size diffs
  • background: 0 Bytes (0.00%)
  • ui: 0 Bytes (0.00%)
  • common: 0 Bytes (0.00%)

Copy link
Copy Markdown
Member

@Gudahtt Gudahtt left a comment

Choose a reason for hiding this comment

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

LGTM!

@DDDDDanica DDDDDanica merged commit fbd9bbd into develop Jun 11, 2024
@DDDDDanica DDDDDanica deleted the fix/deprecated-tag branch June 11, 2024 19:11
@github-actions github-actions bot locked and limited conversation to collaborators Jun 11, 2024
@metamaskbot metamaskbot added the release-12.1.0 Issue or pull request that will be included in release 12.1.0 label Jun 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

release-12.1.0 Issue or pull request that will be included in release 12.1.0 team-extension-platform Extension Platform team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants