AMM Freeze Authorities #39
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…y working, also removed some deprecated structs and code and left better notations on the others
bhgames
commented
Mar 29, 2021
| SWAP_PROGRAM_OWNER_FEE_ADDRESS, | ||
| WRAPPED_SOL_MINT, | ||
| } from "./ids"; | ||
| import { Account, AccountInfo, Connection, PublicKey, SystemProgram, TransactionInstruction } from '@solana/web3.js'; |
There was a problem hiding this comment.
This file got auto formatted by our new pretty standards, so Im going to mark with comments where to look for actual changes.
bhgames
commented
Mar 29, 2021
| } else { | ||
| result.data = layout.decode(item.account.data); | ||
| let layout; | ||
| switch (item.account.data.length) { |
There was a problem hiding this comment.
here, switched to case statement
bhgames
commented
Mar 29, 2021
| result.data = layout.decode(item.account.data); | ||
| let pool = toPoolInfo(result, swapId); | ||
| pool.legacy = isLegacy; | ||
| poolsArray.push(pool as PoolInfo); |
bhgames
commented
Mar 29, 2021
| poolsArray.push(pool as PoolInfo); | ||
| } | ||
| let pool = toPoolInfo(result, swapId); | ||
| pool.legacy = isLegacy; |
There was a problem hiding this comment.
see this code is unindented now because the if/else is gone
bhgames
commented
Mar 29, 2021
| }); | ||
| } | ||
|
|
||
| export async function setFreezeAuthorityBitMask( |
There was a problem hiding this comment.
This is a new action that isn't hooked up anywhere in the UI but I used it with a transient button to successfully integration test the functionality in the contract. Can be hooked up to a UI later.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Necessary changes to have freeze bit mask working and freeze authority working, also removed some deprecated structs and code and left better notations on the others.
Relates to solana-labs/solana-program-library#1515
The new prettier standards we use (120 length lines + ' quotes) caused reformatting in the files I touched, and while this isn't a problem in the smaller ones, there was one large one that got touched (pools.tsx) that caused a lot of noisy changes that aren't real. For ease of reading, I notated the ACTUAL changes I made with comments so please scroll through the file to find those. I notated at the beginning of the long file that this is so.