Skip to content

OKO-631#279

Merged
eldenpark merged 18 commits intodevelopfrom
OKO-631
Feb 23, 2026
Merged

OKO-631#279
eldenpark merged 18 commits intodevelopfrom
OKO-631

Conversation

@chihunmanse
Copy link
Copy Markdown
Contributor

Pull Request

Thank you for raising a Pull Request. Please follow the instruction.

  • I’ve read CONTRIBUTING.md and followed the guidelines.

Summary

Links (Issue References, etc, if there's any)

chihunmanse and others added 16 commits February 13, 2026 14:06
email_template_2: add oko wallet email template
* oko_attached: Use CSS gradients for Base sponsorship background

* oko_attached: Attach tooltip to icon wrapper and adjust styles

* oko_attached: Use tertiary background for sponsored fee tooltip

* oko_attached: prevent sponsored fee UI from showing when balance is
sufficient for fee

* oko_attached: fix fee calculation to use actual gas estimation and wait
for L1 fee on OP Stack chains

* oko_attached: hide next free transaction timer during signing and show
keep window open message on success

* oko_attached: Refetch fee sponsorship query every 10s

* oko_attached: Refetch status on fee sponsorship error

* oko_attached: Clamp remainingMs to not increase on update

* oko_attached: Move tx receipt waiting into sponsorship flow

* oko_attached: Use CSS classes for sponsorship backgrounds

* oko_attached: Use viem Hex type for tx hash

use crate::sss::Point256;
use crate::sss::ReshareResult;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Unnecessary empty line

secret: [u8; 32],
ks_node_hashes: Vec<[u8; 32]>,
t: u32,
) -> Result<Vec<Point256>, String> {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Could we define an Error enum?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@chihunmanse I'll handle it

}

/// Recover the secret from t shares via Lagrange interpolation at x=0.
pub fn seed_combine(split_points: Vec<Point256>, t: u32) -> Result<[u8; 32], String> {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Error, please see the above comment. Also check the same use of this Error enum in this file

@eldenpark eldenpark merged commit 25273d3 into develop Feb 23, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants