Skip to content

Conversation

makemesteaks
Copy link
Member

Description

Changelog

CHANGELOG entry:

Related issues

Fixes:

Manual testing steps

Feature: my feature name

  Scenario: user [verb for user action]
    Given [describe expected initial app state]

    When user [verb for user action]
    Then [describe expected outcome]

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.

@makemesteaks makemesteaks requested a review from a team as a code owner September 12, 2025 09:28
@metamaskbot metamaskbot added the team-dev-ops DevOps team label Sep 12, 2025
STAKING_CLAIM = 'staking_claim',
TRANSFER = 'transfer',
}
// test
Copy link

Choose a reason for hiding this comment

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

Bug: Unintended Debugging Comment Committed

A temporary // test debugging comment, unrelated to the PR's purpose, was accidentally committed to app/core/Analytics/events/confirmations/constants.ts.

Fix in Cursor Fix in Web

Copy link

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants