Skip to content
This repository has been archived by the owner on Jan 24, 2024. It is now read-only.

feat: add pirex gmx tracking #2243

Merged
merged 2 commits into from
Feb 9, 2023
Merged

feat: add pirex gmx tracking #2243

merged 2 commits into from
Feb 9, 2023

Conversation

swheel33
Copy link
Contributor

@swheel33 swheel33 commented Feb 3, 2023

Description

Add tracking for new px wrapper tokens and vault autocompounders with the new pirex gmx update. Tokens added are pxGMX, pxGLP, apxGMX, and apxGLP.

Checklist

  • I have followed the Contributing Guidelines
  • (optional) As a contributor, my Ethereum address/ENS is: hotwheels.dev
  • (optional) As a contributor, my Twitter handle is: @0xhotwheels

How to test?

pxGMX: 0x9a592b4539e22eeb8b2a3df679d572c7712ef999
pxGLP: 0x0eac365e4d7de0e293078bd771ba7d0ba9a4c892
apxGMX: 0x61dbc0d6d7a6bde37f93d2014044a7c6b6de34d2
apxGLP: 0x6cec1903eae091bdc532cfde68fff0723fd3ee14

@swheel33 swheel33 changed the title feat(Pirex): Pirex GMX Updates feat: add pirex gmx tracking Feb 3, 2023
Copy link
Contributor

@wpoulin wpoulin left a comment

Choose a reason for hiding this comment

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

This is a suggestion which is optional, but because ArbitrumPirexApxgmxTokenFetcher and ArbitrumPirexApxglpTokenFetcher share the same logic, you could either extract the shared logic or have both in the same app token using definitions.

I'll approve it as soon as you've defined groupLabels for both app tokens

@wpoulin wpoulin merged commit 8766bdf into Zapper-fi:main Feb 9, 2023
0xdapper pushed a commit to 0xdapper/studio that referenced this pull request Feb 28, 2023
* feat: add pirex gmx tracking

* fix: add group labels
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants