Skip to content

Safe allowlist minting #38

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

Merged
merged 3 commits into from
Apr 3, 2025
Merged

Safe allowlist minting #38

merged 3 commits into from
Apr 3, 2025

Conversation

pheuberger
Copy link
Member

@pheuberger pheuberger commented Mar 24, 2025

Pass overrides.safeAddress to the claimFractionFromAllowlist() method to submit the transaction to the Safe app.

This is using the same pattern as the mintHypercert() method.

Example transaction created using this code: https://app.safe.global/transactions/tx?safe=sep:0x6C622Ca31a755b4A543f1FE1dB26A666A468Fda8&id=multisig_0x6C622Ca31a755b4A543f1FE1dB26A666A468Fda8_0x4ceecae4047d731de541d672b864b5ef930a845df2e79713231f43cbdf974cd8

Copy link

github-actions bot commented Mar 24, 2025

Coverage Report

Status Category Percentage Covered / Total
🟢 Lines 78.97% (🎯 78%) 1157 / 1465
🟢 Statements 78.97% (🎯 78%) 1157 / 1465
🟢 Functions 78.44% (🎯 77%) 91 / 116
🟢 Branches 85.04% (🎯 85%) 273 / 321
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
src/client.ts 70.02% 81.91% 55.55% 70.02% 81-83, 94-95, 105-106, 131-133, 177-209, 215-216, 220-221, 238-239, 251-257, 270-273, 288-298, 313-323, 329-336, 384-385, 428-429, 463-464, 507-513, 523-530, 598-600, 675-676
src/safe/SafeTransactions.ts 100% 92.3% 100% 100%
Generated in workflow #32 for commit 17b89bf by the Vitest Coverage Report Action

The name of the method was too specific since it's a fairly generic
method. It will be used by the claim function that is going to be
introduced in the next commit.
Akin to the mintHypercert method, enable the
claimFractionFromAllowlist() method to follow the same pattern.
There were some uncovered lines in the constructor.
@pheuberger pheuberger merged commit 6d8641c into main Apr 3, 2025
2 checks passed
@pheuberger pheuberger deleted the safe-allowlist-minting branch April 3, 2025 15:04
@bitbeckers
Copy link
Contributor

🎉 This PR is included in version 2.7.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants