Skip to content
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

Threshold moved to Multisig workbook #47

Merged
merged 1 commit into from
Feb 14, 2023
Merged

Conversation

pnowosie
Copy link
Contributor

@pnowosie pnowosie commented Feb 9, 2023

Generator is now taking Multisig threshold from Multisig tab, this prevents some msig definitions were skipped from json because its control group was not used in any allocation.
There is new column Title in the deposit offers tab to be used as offer's description (memo)

It also delivers refactoring to the parsing structures:

  • load_data file is moved to the workbook module and provides some common to parsing structures
  • DTO structures with captures parsed row from xls have better name and shares the same interface
  • 1st row (header) of file is detected and skipped, any further error in parsing would cause generation crash (no skipping errors)

@pnowosie pnowosie marked this pull request as ready for review February 13, 2023 10:46
@pnowosie pnowosie force-pushed the threshold-in-msig-table branch 2 times, most recently from 0b1dbd6 to 79b5b02 Compare February 14, 2023 08:23
- threshold moved to MSig tab,
- new deposit offer title column (memo),
- refactoring and more strict error handling
@pnowosie pnowosie merged commit dac0d3f into dev Feb 14, 2023
@pnowosie pnowosie deleted the threshold-in-msig-table branch February 14, 2023 09:19
c4t-ag pushed a commit that referenced this pull request Feb 24, 2023
- threshold moved to MSig tab,
- new deposit offer title column (memo),
- refactoring and more strict error handling
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants