Skip to content

(fix): Rename LbInfo interface#267

Open
aspnxdd wants to merge 2 commits into
MeteoraAg:mainfrom
aspnxdd:fix/rename-lb-info
Open

(fix): Rename LbInfo interface#267
aspnxdd wants to merge 2 commits into
MeteoraAg:mainfrom
aspnxdd:fix/rename-lb-info

Conversation

@aspnxdd

@aspnxdd aspnxdd commented Jan 22, 2026

Copy link
Copy Markdown

Rename PositionInfo to LbInfo, as this interface references the Lb (Pool) data.

@aspnxdd

aspnxdd commented Mar 17, 2026

Copy link
Copy Markdown
Author

In the class DLMM, method getAllLbPairPositionsByUser:

if (positionData) {
        positionsMap.set(lbPair.toBase58(), {
          publicKey: lbPair,
          lbPair: lbPairAcc,
          tokenX,
          tokenY,
          lbPairPositionsData: [
            ...(positionsMap.get(lbPair.toBase58())?.lbPairPositionsData ?? []),
            {
              publicKey: positionPubkey,
              positionData,
              version,
            },
          ],
        });
      }

The publicKey refers to the lbPair, not the position pubkey. This suggests that the interface refers to a Lb, not a position.

@aspnxdd aspnxdd changed the title Rename LbInfo interface (fix): Rename LbInfo interface Mar 17, 2026
@aspnxdd

aspnxdd commented Mar 17, 2026

Copy link
Copy Markdown
Author

@dannxbt just tagging you for visibility!

FYI It should also be renamed in python files (class PositionInfo():) , .md files and in some comments, I can do that as well.

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.

1 participant