Fix a few data types in the config docs

[chrisbygrave]

• fix a few data types that aren't correct in config.md
• update contributors URL - dead link to the (I believe) correct link
• gas oracle modes - fixed is referenced in a few examples but wasn't included in the list of supported values

Signed-off-by: Chris Bygrave [email protected]

[peterbroadhurst]

Hi @chrisbygrave, this is great!
However... these types are actually built into Markdown from source, and the build checks what's checked in is matched with the latest generation.
You have to run make reference before committing, otherwise the make build fails showing you the mismatch.

So they need to be changed where the parameters are defined - which in this case is actually in some pre-reqs.
I've put in some pointers, and there's one that's confused me as to how we're getting the bug you point out.

[chrisbygrave]

Thanks for the pointers @peterbroadhurst! I hadn't spotted this was generated - I'll work through the pointers in a couple of follow-on PRs

[chrisbygrave]

Ok, so:

• this PR simply fixes the URL in the CONTRIBUTING.md
• Add 'fixed' to the enum options for gas oracle mode firefly-transaction-manager#39 fixes the gas oracle mode
• Fix a few data types firefly-common#36 fixes the factor data type, and also the string array data types

Searching across hyperledger, I didn't find anywhere else using the FloatType other than firefly-transaction-manager, which uses it as the type for the event streams retry factor - another place we wanted to fix.

[peterbroadhurst]

ok - release up for common here, that'll need to go into the FFTM PR

[codecov-commenter]

Codecov Report

Merging #36 (3e9b8bd) into main (4767448) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #36   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           22        22           
  Lines         1293      1291    -2     
=========================================
- Hits          1293      1291    -2     

Impacted Files	Coverage Δ
internal/ethereum/event_listener.go	100.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[chrisbygrave]

I'll update this one once accordingly we get the FFTM PR through

[peterbroadhurst]

https://github.com/hyperledger/firefly-transaction-manager/releases/tag/v1.1.1 ready to pull in here now @chrisbygrave

[peterbroadhurst]

Thanks @chrisbygrave for working this through all the layers 💪