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

REL-004: expiration mechanism #102

Merged
merged 1 commit into from
Dec 7, 2022
Merged

REL-004: expiration mechanism #102

merged 1 commit into from
Dec 7, 2022

Conversation

ironFe93
Copy link
Contributor

@ironFe93 ironFe93 commented Nov 28, 2022

What

  • add 'validUntilTime' field in forward and deploy request

Why

https://rsklabs.atlassian.net/browse/PP-583

Refs

Unit tests inluded in this PR: #104

@ironFe93 ironFe93 marked this pull request as draft November 28, 2022 15:27
@ironFe93 ironFe93 marked this pull request as ready for review November 30, 2022 14:07
@ironFe93 ironFe93 requested review from a team and ulasanil and removed request for ulasanil December 1, 2022 11:17
fix: missing test parameter, missing compilation

feat: add encode params

fix: tests

fix: add expiration parameter to tests

fix: add missing parameters to TypedRequestData.ts

fix: attempt to fix tests

fix: add missing encodes

fix: number to string

fix: missing parameter on test

fix: remove parameter from fctory contract

fix: request param string to number

Revert "fix: request param string to number"

This reverts commit db690fe.

feat: implement check on wallet deployment

chore: update package version
@antomor antomor added the enhancement New feature or request label Dec 7, 2022
@antomor antomor requested a review from ulasanil December 7, 2022 17:08
Copy link
Collaborator

@antomor antomor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@antomor antomor merged commit 863d4a7 into master Dec 7, 2022
@antomor antomor deleted the PP-583/REL-004 branch December 7, 2022 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants