Skip to content

Conversation

@AntonioVentilii
Copy link
Collaborator

Motivation

For the next release, we prefer not to include the OISY rewards ICRC token (staging) yet. We will re-include it later.

Copilot AI review requested due to automatic review settings November 19, 2025 09:23
@AntonioVentilii AntonioVentilii requested a review from a team as a code owner November 19, 2025 09:23
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes the test OISY rewards ICRC token from staging in preparation for the next release. The token will be re-included in a future release.

  • Removed "testOISY.P" token configuration from the ICRC tokens JSON file
  • Removed TESTOISYP_LEDGER_CANISTER_ID constant definition and usage from testnet IDs array

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/frontend/src/env/tokens/tokens.icrc.json Removed testOISY.P token configuration entry
src/frontend/src/env/networks/networks.icrc.env.ts Removed TESTOISYP_LEDGER_CANISTER_ID constant and its reference in ICRC_LEDGER_CANISTER_TESTNET_IDS array

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Collaborator

@sbpublic sbpublic left a comment

Choose a reason for hiding this comment

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

lgtm

@AntonioVentilii AntonioVentilii added this pull request to the merge queue Nov 19, 2025
Merged via the queue into main with commit 860bcee Nov 19, 2025
85 checks passed
@AntonioVentilii AntonioVentilii deleted the feat-frontend/Remove-test-OISY-rewards-ICRC-token branch November 19, 2025 09:34
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.

2 participants