Fixes on raised issues #282
Open
blurbeast wants to merge 3 commits into
Open
Conversation
…points - event_manager: validate_ticket_price now also rejects negative prices on the legacy create_event path; restored proper organizer-tracking and waitlist-promotion helpers that a prior merge had stubbed out. - marketplace: bound multipliers in __constructor and update_price_cap; reject non-positive price and negative token_id in create_listing (now Result-typed); defense-in-depth price guard in purchase_ticket; hard cap and saturating arithmetic on get_active_listings pagination. - ticket_nft: bound name/description/image/tier/uri/event_name strings on update_metadata, update_off_chain_uri, and register_event. - fuzz/integration tests: add missing testutils::Ledger import; switch fuzz to try_create_event_v2; use Address::generate for fuzz admin. - .gitignore: ignore test_snapshots/ and proptest-regressions/ test artifacts. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…ement create_profile, update_profile, and get_profile let a user own a single profile keyed by their Soroban Address. Strings (username, display_name, bio, avatar_uri) are bounds-checked on every write; required fields (username, display_name) reject empty input; updates are partial via Option<String> per field. has_profile is a read-only existence check. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Standalone library crate exposing pause / unpause / is_paused / require_not_paused. Authorization is the host contract's concern, so the module can be reused regardless of how the contract gates admin access. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
@blurbeast Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
Contributor
|
you got crazy conflicts, pull before making a pr |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
in this pr , it resolves some issues tagged below
closes #240
closes #186
closes #180