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.

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

foundation for MUD workshop #1

Closed
wants to merge 9 commits into from
Closed

foundation for MUD workshop #1

wants to merge 9 commits into from

Conversation

alvrs
Copy link
Member

@alvrs alvrs commented Sep 26, 2024

Changes to the original builder example to create a foundation for the MUD workshop.

List of changes:

  • Set up pnpm dev script that forks the chain running in docker, so we can easily and quickly reset the development chain to the state right after deploying the EVE contracts
  • Add SmartTurretTest.t.sol to easily test logic in SmartTurretSystem.sol
  • Add filterPriorityQueue and pushPriorityQueue utils to create foundation for a simple access list functionality
  • TODO: Minimal client setup with store sync, burner wallet, world contract

Changes to do during the workshop:

  • Create GuestList table, modify inProximity function to read from the table
  • Add a new system to add ids to the guest list
  • Access table data in the client, add a button to add ids to the guest list

@@ -4,18 +4,12 @@ procs:
shell: pnpm run dev
contracts:
cwd: packages/contracts
shell: pnpm mud dev-contracts --rpc http://127.0.0.1:8545
shell: pnpm mud dev-contracts --rpc http://127.0.0.1:1234 --worldAddress 0x8a791620dd6260079bf849dc5567adc3f2fdc318

Choose a reason for hiding this comment

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

This is so cool, you can fork the chain running in the docker.

@alvrs
Copy link
Member Author

alvrs commented Sep 27, 2024

Replaced by #2 for workshop foundation and #3 for final workshop state

@alvrs alvrs closed this Sep 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants