Add comments to CR config

smartcontractkit · Feb 12, 2025 · efcba22 · efcba22
1 parent 7a2b323
commit efcba22
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/core/capabilities/ccip/configs/solana/contract_reader.go b/core/capabilities/ccip/configs/solana/contract_reader.go
@@ -69,11 +69,13 @@ func DestContractReaderConfig() (config.ContractReader, error) {
 							},
 							&codec.HardCodeModifierConfig{
 								OnChainValues: map[string]any{
+									// Also doesn't exist on Solana?
+									"NonceManager": solana.PublicKey{},
+									// I don't see this being used in the ccip plugin, but it's still worth figuring out what the equivalent is
+									"TokenAdminRegistry": solana.PublicKey{},
+									// these two don't exist on Solana
 									"GasForCallExactCheck": 0,
 									"RmnRemote":            solana.PublicKey{},
-									// TODO what to do with these two? Do they share address with router?
-									"TokenAdminRegistry": solana.PublicKey{},
-									"NonceManager":       solana.PublicKey{},
 								},
 							},
 						},
@@ -162,7 +164,7 @@ func DestContractReaderConfig() (config.ContractReader, error) {
 							},
 						},
 					},
-					// TODO this one is hacky, NONEVM-1320
+					// This one is hacky, but works - [NONEVM-1320]
 					consts.MethodNameFeeQuoterGetTokenPrices: {
 						ChainSpecificName: "BillingTokenConfigWrapper",
 						PDADefinition: solanacodec.PDATypeDef{