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

feat(slot-reservations): support enforcement of slot reservations before filling slot #934

Merged
merged 1 commit into from
Oct 9, 2024

Conversation

emizzle
Copy link
Contributor

@emizzle emizzle commented Oct 3, 2024

Closes #933.

This is being implemented as part of a longer list of tasks for slot reservations to prevent PRs from being too large.

The contracts changes included in this PR enforce that a slot is reserved before it is filled.

This PR adds support for these updates by updating the tests to reserve slots before filling them.

Copy link
Member

@markspanbroek markspanbroek left a comment

Choose a reason for hiding this comment

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

👍

@emizzle emizzle force-pushed the feat/slot-reservations/4-add-slotreservationfull-event branch from 9fb5326 to d6d2e72 Compare October 5, 2024 00:10
Base automatically changed from feat/slot-reservations/4-add-slotreservationfull-event to master October 5, 2024 02:47
@emizzle
Copy link
Contributor Author

emizzle commented Oct 5, 2024

@benbierens with this change in, a fillSlot transaction will require a reserveSlot transaction first. I'm not sure if it affects your distributed tests or not so just giving you the heads up.

@emizzle emizzle force-pushed the feat/slot-reservations/6-enforce-slot-reservations branch from dd5a31c to 7fb8e77 Compare October 8, 2024 05:06
@emizzle emizzle added this pull request to the merge queue Oct 9, 2024
Merged via the queue into master with commit 7e0ec3c Oct 9, 2024
11 checks passed
@emizzle emizzle deleted the feat/slot-reservations/6-enforce-slot-reservations branch October 9, 2024 05:56
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.

feat(slot-reservations): support enforcement of reserving slots before filling
3 participants