Skip to content

feat: adds expiry to proposal payload#193

Open
ganchoradkov wants to merge 3 commits intomain-lfs-DO-NOT-USEfrom
expiry-in-proposal
Open

feat: adds expiry to proposal payload#193
ganchoradkov wants to merge 3 commits intomain-lfs-DO-NOT-USEfrom
expiry-in-proposal

Conversation

@ganchoradkov
Copy link
Member

No description provided.

@vercel
Copy link

vercel bot commented Jan 15, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
walletconnect-specs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 18, 2024 1:04pm

},
"pairingTopic": string
"pairingTopic": string,
"expiry": number, // OPTIONAL timespamp in seconds
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"expiry": number, // OPTIONAL timespamp in seconds
"expiry": number, // OPTIONAL time Interval Since 1970 in seconds```

},
"pairingTopic": string
"pairingTopic": string,
"expiryTimestamp": number, // OPTIONAL timespamp in seconds
Copy link
Contributor

Choose a reason for hiding this comment

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

We should add expiryTimestamp on json rpc section for session proposal and session request

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 this pull request may close these issues.

3 participants