opened
catenocrypt
## Description

Refactor the generated Coins.cpp files, instead of independent methods with large switch statements, create a map with all coin data, and retrieve data from there.
Motivation: data is closer to the coins.json file.  Solves in part #885 .

## Testing instructions

Build, check `src/Generated/Coins.cpp`.

Unit tests.

## Types of changes

* Minor internal refactor

## Checklist

<!--- The following points should be used to indicate the progress of your PR.  Put an `x` in all the boxes that apply right now, and come back over time and check them off as you make progress.  If you're unsure about any of these, don't hesitate to ask. We're here to help! -->

- [x] Prefix PR title with `[WIP]` if necessary.
- [x] Add tests to cover changes as needed.
- [x] Update documentation as needed.
- [x] If there is a related Issue, mention it in the description (e.g. 'Fixes #444 ).
[WIP] Refactor Coins.cpp, coin data as a const map
issue
created
catenocrypt
/azp run
comment
created
vikmeup
/azp run
comment
created
vikmeup
/azp run
comment
created
vikmeup
/azp run
comment
created
vikmeup
/azp run
comment
created
vikmeup
/azp run
comment
created
catenocrypt
/azp run
comment
created
catenocrypt
I'm afraid CI teawking is needed.
comment
<samp>
Azure Pipelines successfully started running 3 pipeline(s).<br>

</samp>
This message directly preceded comment from bot *azure-pipelines[bot]* by author *vikmeup*:
/azp run
<samp>
Azure Pipelines successfully started running 3 pipeline(s).<br>

</samp>
This message directly preceded comment from bot *azure-pipelines[bot]* by author *catenocrypt*:
/azp run
This message directly preceded comment from bot *azure-pipelines[bot]* by author *vikmeup*:
/azp run
This message directly preceded comment from bot *azure-pipelines[bot]* by author *catenocrypt*:
/azp run