Skip to content

Release/1.1.0 #75

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.

Sign up for GitHub

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

Jump to bottom
Closed
Le-Caignec wants to merge 146 commits into from
Closed

Release/1.1.0 #75

Le-Caignec wants to merge 146 commits into from

Conversation

@Le-Caignec
Copy link
Contributor

@Le-Caignec Le-Caignec commented Mar 7, 2025

No description provided.

james-toussaint and others added 30 commits September 20, 2024 17:46
@james-toussaint
Add dealId to TaskClaimedWithVoucher event
c4f3454
@james-toussaint
Index args in events
ee8f6a6
@james-toussaint
Initialize fresh vNEXT section in changelog
a39e172
@james-toussaint
Back merge 1.0.0 (#48)
e79a869
@gfournierPro
migrate blockscout v6 dns
ed33fb9
@gfournierPro
remove blockscout v5 refs
b818b16
@gfournierPro
update changelog
a9c7301
@gfournierPro
Remove blockscout v5 refs (#49)
9d332af
@Le-Caignec
fix token decimal & add comment for Halborn audit
363b65b
@Le-Caignec
add one line into changelog
61bf8de
@zguesmi
Verify VoucherProxy contracts
9a54e87
@zguesmi
Verify VoucherProxy contracts (#52)
145178d
@Le-Caignec
add new script
53abb08
@Le-Caignec
add one new test for the decimals function
18c2224
@Le-Caignec
Merge remote-tracking branch 'origin/develop' into feature/patch-toke…
e2a0baa 
…n-decimal-voucherHub
@Le-Caignec @zguesmi
Update CHANGELOG.md
f8c6408 
Co-authored-by: Zied Guesmi <26070035+zguesmi@users.noreply.github.com>
@Le-Caignec @zguesmi
Update contracts/VoucherHub.sol
c04dadd 
Co-authored-by: Zied Guesmi <26070035+zguesmi@users.noreply.github.com>
@Le-Caignec @zguesmi
Update test/VoucherHub.test.ts
eb67939 
Co-authored-by: Zied Guesmi <26070035+zguesmi@users.noreply.github.com>
@Le-Caignec
move test into NonTransferableERC20Upgradeable describe section
ce51662
@Le-Caignec
Refactor decimals function documentation for clarity and consistency
699149a
@Le-Caignec
Refactor environment variable handling to use a centralized env module
f5d02f9
@Le-Caignec
Update package dependencies: add dotenv and zod to devDependencies
e3dff53
@Le-Caignec
Refactor deployment script to use environment variable for local fork…
80d599d 
… check
@Le-Caignec
Update CHANGELOG.md to include reference of the PR
8675176
@Le-Caignec
Clarify documentation for updateVoucherTypeDescription to indicate in…
52180cb 
…tended feature per Halborn audit
@Le-Caignec
reorder functions into the NonTransferableERC20Upgradeable contract t…
06e7115 
…o stay consistent with the standard ERC20 order
@Le-Caignec
Improve the documentation of updateVoucherTypeDescription function
c9ee780
@Le-Caignec
Feature/patch token decimals into voucher hub (#50)
224fe6c
@Le-Caignec @zguesmi
Update .env.template
8d3de1e 
Co-authored-by: Zied Guesmi <26070035+zguesmi@users.noreply.github.com>
@Le-Caignec @zguesmi
Update .env.template
eb74a71 
Co-authored-by: Zied Guesmi <26070035+zguesmi@users.noreply.github.com>
@Le-Caignec Le-Caignec self-assigned this Mar 7, 2025
@Le-Caignec Le-Caignec requested review from gfournierPro, james-toussaint and zguesmi and removed request for zguesmi March 7, 2025 10:59
gfournierPro
gfournierPro approved these changes Mar 7, 2025
View reviewed changes
Copy link
Contributor

@gfournierPro gfournierPro left a comment

Choose a reason for hiding this comment

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

Thanks :)

@codecov
Copy link

codecov bot commented Mar 18, 2025 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.30%. Comparing base (3e26775) to head (ec33f87).
Report is 14 commits behind head on main.

Additional details and impacted files 
@@           Coverage Diff           @@
##             main      #75   +/-   ##
=======================================
  Coverage   99.29%   99.30%           
=======================================
  Files           4        4           
  Lines         284      286    +2     
  Branches       34       34           
=======================================
+ Hits          282      284    +2     
  Misses          2        2

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

zguesmi
zguesmi reviewed Mar 18, 2025
View reviewed changes
@gfournierPro gfournierPro deleted the release/1.1.0 branch March 18, 2025 10:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@zguesmi zguesmi zguesmi approved these changes

@james-toussaint james-toussaint Awaiting requested review from james-toussaint

+1 more reviewer

@gfournierPro gfournierPro gfournierPro approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@Le-Caignec Le-Caignec

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@Le-Caignec @zguesmi @gfournierPro @james-toussaint