Skip to content

Conversation

@MoMannn
Copy link
Contributor

@MoMannn MoMannn commented Oct 15, 2025

Description

Replace header and sub header with more meaningful text.

Manual testing steps

  1. Go to http://localhost:8000/
  2. Grant permission

Screenshots/Recordings

Before

Screenshot 2025-10-15 at 11 51 21

After

Screenshot 2025-10-15 at 11 49 25

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.

Note

Standardizes permission titles: periodic -> "Token Subscription" and stream -> "Token Stream" for both ERC20 and native tokens.

  • Permissions:
    • Rename title to "Token Subscription" in:
      • packages/gator-permissions-snap/src/permissions/erc20TokenPeriodic/index.ts
      • packages/gator-permissions-snap/src/permissions/nativeTokenPeriodic/index.ts
    • Rename title to "Token Stream" in:
      • packages/gator-permissions-snap/src/permissions/erc20TokenStream/index.ts
      • packages/gator-permissions-snap/src/permissions/nativeTokenStream/index.ts

Written by Cursor Bugbot for commit ced844f. This will update automatically on new commits. Configure here.

@MoMannn MoMannn requested a review from a team as a code owner October 15, 2025 10:17
Copy link
Contributor

@jeffsmale90 jeffsmale90 left a comment

Choose a reason for hiding this comment

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

The world smallest nit!

@MoMannn MoMannn requested a review from jeffsmale90 October 20, 2025 13:31
Copy link
Contributor

@jeffsmale90 jeffsmale90 left a comment

Choose a reason for hiding this comment

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

Let's go!

@MoMannn MoMannn merged commit 8b714c6 into main Oct 21, 2025
16 checks passed
@MoMannn MoMannn deleted the chore/rename-header branch October 21, 2025 07:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants