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

initial towns token #1990

Open

giuseppecrj wants to merge 18 commits into main from g/token

+1,214 −74

Contributor

giuseppecrj commented Jan 8, 2025

No description provided.


          initial towns token

d2e87d8

vercel bot commented Jan 8, 2025 •

edited

Loading

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

1 Skipped Deployment

Name	Status	Preview	Comments	Updated (UTC)
river-sample-app	⬜️ Ignored (Inspect)	Visit Preview		Jan 14, 2025 10:45pm

giuseppecrj added 5 commits

January 8, 2025 17:08


          initial tests for towns token

8645f6c


          couple more tests

a50e33a


          update solady erc20 with new changes

af58417


          towns base

a3b9b21


          disable initializer

d8a519f

giuseppecrj marked this pull request as ready for review

January 10, 2025 21:40

giuseppecrj requested review from pfives, jterzis, shuhuiluo and texuf as code owners

January 10, 2025 21:40

giuseppecrj added 2 commits

January 10, 2025 16:40


          Merge branch 'main' into g/token

7b1a38e


          add more getters

9df5662

shuhuiluo requested changes

View reviewed changes

contracts/src/tokens/towns/base/Towns.sol Show resolved Hide resolved

contracts/src/tokens/towns/base/Towns.sol Show resolved Hide resolved

contracts/src/tokens/towns/base/Towns.sol Show resolved Hide resolved

shuhuiluo requested changes

View reviewed changes

contracts/src/tokens/towns/base/Towns.sol Outdated Show resolved Hide resolved

contracts/src/tokens/towns/base/Towns.sol Outdated Show resolved Hide resolved

contracts/src/tokens/towns/base/Towns.sol Outdated Show resolved Hide resolved

contracts/src/tokens/towns/base/Towns.sol Outdated Show resolved Hide resolved

contracts/src/tokens/towns/base/Towns.sol Outdated Show resolved Hide resolved

shuhuiluo requested changes

View reviewed changes

contracts/src/tokens/towns/base/Towns.sol Outdated Show resolved Hide resolved

contracts/src/tokens/towns/mainnet/libs/VotesEnumerableLib.sol Outdated Show resolved Hide resolved


          updates based on pr review

9a4460c

shuhuiluo requested changes

View reviewed changes

contracts/src/tokens/towns/mainnet/libs/VotesEnumerableLib.sol Outdated Show resolved Hide resolved

contracts/src/tokens/towns/mainnet/libs/TokenInflationLib.sol Outdated Show resolved Hide resolved

contracts/src/tokens/towns/mainnet/libs/TokenInflationLib.sol Outdated Show resolved Hide resolved

shuhuiluo requested changes

View reviewed changes

contracts/test/tokens/towns/TownsMainnet.t.sol Show resolved Hide resolved


          get paginated delegators

67c510e

shuhuiluo requested changes

View reviewed changes

contracts/src/tokens/towns/mainnet/libs/TokenInflationLib.sol Outdated Show resolved Hide resolved

giuseppecrj added 2 commits

January 13, 2025 16:34


          fix inflation rate calculation


          create inflation fuzz

c09ac01

giuseppecrj requested a review from shuhuiluo

January 13, 2025 22:17

shuhuiluo requested changes

View reviewed changes

contracts/test/tokens/towns/TownsMainnet.t.sol Outdated Show resolved Hide resolved

contracts/src/tokens/towns/mainnet/libs/VotesEnumerableLib.sol Outdated Show resolved Hide resolved

shuhuiluo requested changes

View reviewed changes

contracts/src/tokens/towns/mainnet/libs/VotesEnumerableLib.sol Outdated Show resolved Hide resolved


          updates based on PR review

f8a78b5

giuseppecrj requested a review from shuhuiluo

January 14, 2025 06:58

shuhuiluo requested changes

View reviewed changes

contracts/src/diamond/facets/governance/votes/enumerable/VotesEnumerable.sol Outdated Show resolved Hide resolved

contracts/src/tokens/towns/mainnet/Towns.sol Outdated Show resolved Hide resolved

contracts/src/tokens/towns/base/Towns.sol Outdated Show resolved Hide resolved

contracts/src/tokens/towns/base/Towns.sol Show resolved Hide resolved


          remove selector bloating

e61b32c

shuhuiluo requested changes

View reviewed changes

contracts/src/tokens/towns/mainnet/libs/TokenInflationLib.sol Outdated Show resolved Hide resolved

contracts/src/tokens/towns/mainnet/libs/TokenInflationLib.sol Outdated Show resolved Hide resolved

contracts/test/tokens/towns/TownsMainnet.t.sol Outdated Show resolved Hide resolved

contracts/test/tokens/towns/TownsMainnet.t.sol Outdated Show resolved Hide resolved

contracts/src/diamond/facets/governance/votes/enumerable/VotesEnumerable.sol Outdated Show resolved Hide resolved

giuseppecrj added 2 commits

January 14, 2025 11:41


          add more tests and use VoteEnumerable

c183aa6


          use votes enumerable

84730c1

shuhuiluo approved these changes

View reviewed changes

contracts/src/diamond/facets/governance/votes/enumerable/VotesEnumerable.sol Outdated Show resolved Hide resolved

contracts/src/tokens/towns/mainnet/libs/TokenInflationLib.sol Outdated Show resolved Hide resolved

giuseppecrj added 2 commits

January 14, 2025 15:59


          updates based on PR

b7e7a67


          updates to tests

b7f9c9b

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

shuhuiluo shuhuiluo approved these changes

pfives Awaiting requested review from pfives pfives is a code owner

jterzis Awaiting requested review from jterzis jterzis is a code owner

texuf Awaiting requested review from texuf texuf is a code owner

Labels

None yet