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

Remove redundant tx_parameters table. #1727

Closed
1 task
jonathanknowles opened this issue Jun 5, 2020 · 0 comments · Fixed by #1735
Closed
1 task

Remove redundant tx_parameters table. #1727

jonathanknowles opened this issue Jun 5, 2020 · 0 comments · Fixed by #1735

Comments

@jonathanknowles
Copy link
Member

Context

Issue #1693 will introduce a new database table protocol_parameters. This table will include all columns included in the pre-existing tx_parameters table, and add a new decentralization_level column.

The pre-existing tx_parameters table will become redundant.

Decision

Create a startup task to remove the tx_parameters table, if it exists.

Acceptance Criteria

  • The wallet should remove the tx_parameters table on startup, if it exists.

Development

To be written.

QA

To be written.

rvl added a commit that referenced this issue Jun 10, 2020
iohk-bors bot added a commit that referenced this issue Jun 12, 2020
1735: Clean up old TxParameters SQL table r=KtorZ a=rvl

### Issue Number

Closes #1727

### Overview

Remove an old SQL table, if it exists.


Co-authored-by: Rodney Lorrimar <[email protected]>
KtorZ pushed a commit that referenced this issue Jun 12, 2020
iohk-bors bot added a commit that referenced this issue Jun 12, 2020
1735: Clean up old TxParameters SQL table r=KtorZ a=rvl

### Issue Number

Closes #1727

### Overview

Remove an old SQL table, if it exists.


Co-authored-by: Rodney Lorrimar <[email protected]>
iohk-bors bot added a commit that referenced this issue Jun 13, 2020
1735: Clean up old TxParameters SQL table r=rvl a=rvl

### Issue Number

Closes #1727

### Overview

Remove an old SQL table, if it exists.


Co-authored-by: Rodney Lorrimar <[email protected]>
@iohk-bors iohk-bors bot closed this as completed in b263454 Jun 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant