-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Q: how do we know when a new cryptocurrency is added to the SDN? #14
Comments
I'm not actively monitoring for new coins being added to the SDN list - for me personally, only XBT is relevant. However, I accept PRs (#7, #6) to add new coins and will review and merge them. As the CI is already running each night to check if there are new entries for the existing coins, there could also be a job that runs after uploading potential new entries - failing or otherwise alerting that there are new coins. Happy to create an issue for each new coin when the CI fails. Someone (not necessarily me, but I might do it if I have the time) can then create a PR to add the new coin. |
Thank you! |
We might want to keep this open and not "closed as completed". The new CI task I was talking about doesn't exist yet. Someone might want to add that. Reopening with the help wanted label. |
hey i was wondering if this is a related issue, let alone coins, but new addresses, on branch |
The ofac-sanctioned-digital-currency-addresses/.github/workflows/generate-lists.yml Lines 5 to 7 in 319d5a6
|
My guess from reading https://github.com/0xB10C/ofac-sanctioned-digital-currency-addresses/blob/main/generate-address-list.py#L12 is that someone periodically fetches the XML file, does a grep and then compares that to the list at https://github.com/0xB10C/ofac-sanctioned-digital-currency-addresses/blob/main/generate-address-list.py#L13-L15
I am wondering if a better implementation strategy might be to have the script do two passes: first, to look for
<FeatureType/>
starting with "Digital Currency Address - ", to build up the list on lines 13-15, and, second, to proceed as usual. of course, i'd still keep what's on lines 13-15, just as a safety.Your thoughts?
The text was updated successfully, but these errors were encountered: