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

[ALC-5] update initialized from uint8 to uint64 #4

Merged
merged 1 commit into from
Sep 22, 2023

Conversation

jaypaik
Copy link
Collaborator

@jaypaik jaypaik commented Sep 20, 2023

Addresses:
ALC-5: Improvements for the CustomSlotInitializable Contract

@jaypaik jaypaik force-pushed the 09-20-_ALC-2_add_comment_explaining_contract_owner_limitations_for_user_operation_validation branch from a717eca to 60c3919 Compare September 20, 2023 19:22
@jaypaik jaypaik force-pushed the 09-20-_ALC-5_update_initialized_from_uint8_to_uint64 branch from ca6c9f3 to 5ee69ed Compare September 20, 2023 19:22
@jaypaik jaypaik force-pushed the 09-20-_ALC-2_add_comment_explaining_contract_owner_limitations_for_user_operation_validation branch from 60c3919 to 4a6de2e Compare September 20, 2023 19:26
@jaypaik jaypaik force-pushed the 09-20-_ALC-5_update_initialized_from_uint8_to_uint64 branch from 5ee69ed to 9b88fbd Compare September 20, 2023 19:27
@howydev
Copy link
Collaborator

howydev commented Sep 21, 2023

small nit: we're not using it for lightaccount but let's change this to uint64 also to maintain consistency

@howydev howydev force-pushed the 09-20-_ALC-5_update_initialized_from_uint8_to_uint64 branch from 2045691 to bf99b74 Compare September 22, 2023 16:49
Base automatically changed from 09-20-_ALC-2_add_comment_explaining_contract_owner_limitations_for_user_operation_validation to main September 22, 2023 18:09
@howydev howydev merged commit 3b0f9fd into main Sep 22, 2023
2 checks passed
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