Skip to content

refactor: remove ESX purchase confirmation menus - #183

Open
yuu-2k wants to merge 2 commits into
esx-framework:1.17.0from
yuu-2k:remove-esx-purchase-confirmation
Open

yuu-2k wants to merge 2 commits into
esx-framework:1.17.0from
yuu-2k:remove-esx-purchase-confirmation

Conversation

@yuu-2k

@yuu-2k yuu-2k commented Oct 27, 2026

Copy link
Copy Markdown

Description

Removed the ESX purchase confirmation menus (Yes/No) across multiple scripts.

Motivation

The additional confirmation menus were unnecessary and added extra steps to the purchase process. Removing them makes purchases faster and simplifies the user experience.

Implementation Details

  • Removed the extra ESX Yes/No purchase confirmation menus across the affected scripts.
  • Purchases now proceed directly to the existing payment flow when an item or service is selected.
  • Preserved the existing selection menus and other relevant UI functionality.

Usage Example

-- Before: Select an item -> Confirm Yes/No -> PaymentGateway
-- After:  Select an item -> PaymentGateway

PR Checklist

  • My commit messages and PR title follow the Conventional Commits standard.
  • My changes have been tested locally.
  • [x ] My PR does not introduce any breaking changes.
  • I have provided a clear explanation of what my PR does and why.

Note: I've tested the changes on my side. If anyone could test them as well, I'd really appreciate it to make sure everything works as expected.

Yuu2k added 2 commits October 26, 2026 21:33
Remove the ESX Yes/No confirmation menu before purchases.

Open the PaymentGateway directly after selecting a skin or accessory.
Remove the ESX Yes/No confirmation menu before purchases.

Open the PaymentGateway directly.
@N0tNvll
N0tNvll changed the base branch from main to 1.17.0 October 27, 2026 16:51
@N0tNvll
N0tNvll self-requested a review October 27, 2026 16:57
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.

1 participant