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

fix: add missing isUserOpValidation to ValidationDataView in spec #187

Conversation

jaypaik
Copy link
Collaborator

@jaypaik jaypaik commented Sep 30, 2024

This was missing in the spec. Adding it in.

@jaypaik jaypaik requested a review from a team September 30, 2024 21:51
Base automatically changed from 09-30-fix_v0.8_review_and_polish to develop October 1, 2024 20:18
@jaypaik jaypaik force-pushed the 09-30-fix_add_missing_isUserOpValidation_to_ValidationDataView_in_spec branch from 42c537d to d2e8540 Compare October 1, 2024 20:19
Copy link

Summary by Octane

New Contracts

No new contracts were added in this PR.

Updated Contracts

  • IModularAccountView.sol: Updated description clarifies user operation validation functionality.

🔗 Commit Hash: d2e8540

Copy link

Overview

Octane AI analysis has finished. No vulnerabilities were found. Cheers! 🎉🎉🎉


🔗 Commit Hash: d2e8540

@jaypaik jaypaik merged commit 1263a7c into develop Oct 1, 2024
4 checks passed
@jaypaik jaypaik deleted the 09-30-fix_add_missing_isUserOpValidation_to_ValidationDataView_in_spec branch October 1, 2024 20:42
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.

3 participants