[PAY-3380] Skip geo indexing on local dev stack #9544
Conversation
|
| if memo.startswith(GEO_MEMO_STRING): | ||
| if ( | ||
| memo.startswith(GEO_MEMO_STRING) | ||
| and shared_config["discprov"]["env"] != "dev" |
There was a problem hiding this comment.
I think we probably want to more gracefully handle if there isn't geo data even outside of dev - if this throws it makes payment indexing get stuck. Maybe instead of throwing we should log an error and skip the geo info reporting?
[eed6296] Fix include_network_cut parameter (#9555) Marcus Pasell [48b8641] Fix purchasers endpoint (#9549) Andrew Mendelsohn [53deb4b] Remove marshalling for nft-gated-signatures (#9550) Randy Schott [6b170d0] [PAY-3375] get chat endpoint correctly filters for audience:type:id (#9545) Steve Perkins [84f4781] [PAY-3380] Skip geo indexing on local dev stack (#9544) Reed [ba1aaed] Fix trending reward notifs (#9532) Isaac Solo
2 participants
Description
Something changed on the tx we send for purchases, it now includes
geo:nullas the memo data for the geo memo. We could try to not send the memo when performing purchases on local stack, or we could skip the geo step on local stack which is what this PR does. Either way is unclean so figured it doesn't make a difference.How Has This Been Tested?
Local purchase successfully indexed