Skip to content
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

Deploy v1-beta1 #87

Merged
merged 83 commits into from
Jan 26, 2023
Merged

Deploy v1-beta1 #87

merged 83 commits into from
Jan 26, 2023

Conversation

spengrah
Copy link
Member

No description provided.

spengrah and others added 30 commits December 8, 2022 10:17
@spengrah
check if transfer recipient already wears hat
887a940
@spengrah
Merge pull request #75 from Hats-Protocol/fix/transfer-balance-check
fae3e36 
check if transfer recipient already wears hat
@spengrah
check if transfer recipient has hat token balance
0804e99
@spengrah
clean up comments
14d3aa8
@spengrah
Merge pull request #77 from Hats-Protocol/fix/transfer-to-balance-check
9c36a0e 
Fix transfer to balance bug
@spengrah
re-shape hat tree to 14 bits per level (compiling)
022fe72
@spengrah
tests are passing
1fb73d9
@spengrah
update gas snapshot and coverage records
dff61c0
@spengrah
add check and error for invalid child hat
e6fc6f4
@spengrah
update readme
e25f934
@spengrah
prevent transfer of immutable hats
8c068dd
@spengrah
update snapshot and coverage trackers
51fae8e
@spengrah
switch to 14 levels with 16 bits each
60dd6f6
@spengrah
get 14x16 tests passing
be7bf44
@spengrah
reduce optimizer_runs to 10k to ⬇️ contract size
bce9ee6
@spengrah
update coverage and snapshot tracking
a52d839
@spengrah
try update ci to run snapshot and coverage reports
978cc9d
@spengrah
Merge branch 'feat/16x14' into feat/immutable-nontransferability
19b8142
@spengrah
Merge pull request #80 from Hats-Protocol/feat/immutable-nontransfera…
d45c8bd 
…bility

Feat: prevent transfer of immutable hats
@spengrah
add WearerStandingChanged event
586d1c7
@spengrah
check for hat balance in _processHatWearerStatus
523e47d
@spengrah
tests for set/check wearer back in good standing
26baca2
@spengrah
Merge pull request #78 from Hats-Protocol/feat/16x14
4f509e3 
Change hat tree shape to 14 levels with 16 bits per level
@topocount
feat: Allow trees to be linked from the TopHat
11fd945 
This feature allows for the following:
- effectively infinitely deep trees (really, limited to 255 levels)
- migrating suborgs into parent orgs without changing hat Ids

A few todos left:
-  [ ] figure out the naming semantics for linked tophats and other naming
  nuances. Should we just call them "tree roots" when linked?
- [ ] test the new Hats.sol functions
- [ ] Add more compreshensive testing to the HatsIdUtils changes
- [ ] Add test for circular linkages and safe depth
@topocount
Review Comments from @slgraham
96195a7
@topocount
fmt Test Contract
a29290a
@topocount
Add Top Level Linking Tests
cda3d98
@topocount
Updated Comments and cleaned up some details
0eff38b
@topocount
Implement Circular Linkage Check
040b226
@topocount
Add extra circular linking test and hardcode bitshifts
750c3fc 
These bitshifts are cheaper to just hardcode as a single op than throw
in an extra utility and add addtional contextual "jump" opcodes
@spengrah
move circular link check to _linkTopHatToTree
0d72aa7
@spengrah
update coverage and gas tracking
8b36c28
@spengrah
Merge pull request #84 from Hats-Protocol/optimize-v1
c756baa 
Code and gas optimizations for v1
@spengrah
allow newAdminId wearers to approve tree links
a765cea
@spengrah
prevent cross-tree linking
ce23f68
@spengrah
clean up linking names and comments
986b9a0
@spengrah
address test compile warnings
6d9e001
@spengrah
add new link functions to IHats & IHatsIdUtilities
7e569a3
@spengrah
fix admin check for last level hats
eb47136
@spengrah
Merge pull request #85 from Hats-Protocol/feat/consentful-tree-linking
00423a1 
Request and approval flow for tophat links
@spengrah
fill in natspec holes
3cb84ea
@spengrah
add CONTRIBUTING.md
26c1fb2
@spengrah
close more natspec holes and re-organize functions
4b100ef
@spengrah
move all interfaces to /Interfaces
d155e61
@spengrah
add forge fmt to ci & reformat accordingly
6ab2ce5
@spengrah
add deploy workflow for docs
b955799
@spengrah
additional config for auto-generated docs
1fc0e04
@spengrah
allow tophats to change own details & imageURI
5a2f710
@spengrah
update README
a840a88
@spengrah
update snapshot and coverage trackers
161142a
@spengrah
small natspec changes based on @nintynick review
901bf96
@spengrah
gitignore broadcast (deploys viewable in Releases)
994c3c2
@spengrah
Merge pull request #83 from Hats-Protocol/cleanup-v1
287481c 
Further code cleanup and docs
@spengrah
add revert test for non-tophat wearer
7cea2d0
@spengrah
Merge pull request #86 from Hats-Protocol/feat/limited-tophat-mutability
510c606 
Feat: enable tophats to change their own `details` and `imageURI`
@spengrah
deploy v1-beta1 to goerli
36f558b
@spengrah spengrah marked this pull request as ready for review January 26, 2023 02:37
nintynick
nintynick approved these changes Jan 26, 2023
View reviewed changes
Copy link
Member

@nintynick nintynick left a comment

Choose a reason for hiding this comment

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

We reviewed this changes in depth in the develop branch - good to merge.

@spengrah spengrah merged commit 60f07df into main Jan 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nintynick nintynick nintynick approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@spengrah @nintynick @topocount