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: random drops #2486

Merged
merged 6 commits into from
Jan 21, 2025
Merged

Conversation

WeylonSantana
Copy link
Contributor

@WeylonSantana WeylonSantana commented Jan 20, 2025

Resolves #2488

The default Amount for all games now becomes MaxQuantity in the logic of this pr, but nothing has been renamed. So after this pr, all games will drop a minimum of 1 - Amount when they update.

I'm also bringing the possibility of editing items that already exist in the drop list, both in the npcs and resources editor (I don't know why we didn't have this before)

Intersect_Client_impAbCh5M2.mp4


public static LocalizedString dropchance = @"Chance (%):";

public static LocalizedString dropadd = @"Add";

public static LocalizedString dropremove = @"Remove";

public static LocalizedString dropdisplay = @"{00} x{01} - {02}%";
public static LocalizedString dropdisplay = @"{00} x{01} - {02} | {03}%";
Copy link
Member

Choose a reason for hiding this comment

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

Change to {00} x{01}-{02} | {03}%


public static LocalizedString dropchance = @"Chance (%):";

public static LocalizedString dropadd = @"Add";

public static LocalizedString dropremove = @"Remove";

public static LocalizedString dropdisplay = @"{00} x{01} - {02}%";
public static LocalizedString dropdisplay = @"{00} x{01} - {02} | {03}%";
Copy link
Member

Choose a reason for hiding this comment

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

Change to {00} x{01}-{02} | {03}%

Copy link
Member

@lodicolo lodicolo left a comment

Choose a reason for hiding this comment

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

I made a minor QoL change to the add button in frmNpc, it looks like frmResource just needs to be updated to match frmNpc and then this is good to go

@lodicolo lodicolo merged commit 241e419 into AscensionGameDev:main Jan 21, 2025
1 check passed
@lodicolo lodicolo deleted the ranges-drops branch January 21, 2025 19: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.

feature: ability to set amount range for items drop within the NPC editor
2 participants