Skip to content

fix(eloot.lic): v2.7.1 multiple fixes#2216

Open
mrhoribu wants to merge 9 commits intomasterfrom
fix/eloot-v2.7.1
Open

fix(eloot.lic): v2.7.1 multiple fixes#2216
mrhoribu wants to merge 9 commits intomasterfrom
fix/eloot-v2.7.1

Conversation

@mrhoribu
Copy link
Contributor

@mrhoribu mrhoribu commented Jan 28, 2026

  • replaced overflow_container and secondary_overflow with single overflow_containers setting (comma-separated)
  • added configurable "Trash to Dump" section, instead of hardcoded herb/food/junk

Important

Refactor eloot.lic to consolidate overflow settings, add configurable trash dumping, and fix script parameter bug.

  • Settings Refactor:
    • Replaced overflow_container and secondary_overflow with overflow_containers (comma-separated) in eloot.lic.
    • Added trash_dump_types setting for configurable trash dumping.
  • Migration:
    • Added migrate_overflow_settings() to convert old overflow settings to new format.
  • UI and Inventory:
    • Updated UI to reflect new overflow_containers and trash_dump_types settings.
    • Modified inventory handling to dynamically manage overflow containers.
  • Bug Fix:
    • Fixed erroneous commas in script parameters in eloot.lic.

This description was created by Ellipsis for bdb422b. You can customize this summary. It will automatically update as commits are pushed.

    - replaced overflow_container and secondary_overflow with single overflow_containers setting (comma-separated)
    - added configurable "Trash to Dump" section, instead of hardcoded herb/food/junk
ellipsis-dev[bot]

This comment was marked as outdated.

@Deysh
Copy link
Contributor

Deysh commented Jan 28, 2026

Probably want to remove food, herb, and junk toggles in the sell type section since they are covered by the trash to dump section.

@mrhoribu
Copy link
Contributor Author

Probably want to remove food, herb, and junk toggles in the sell type section since they are covered by the trash to dump section.

I kept old functionality that it still checks the "sell" types in order to then trash/junk stuff. So you have to have both checked in order to actually dump. Couldn't think of a better way to safeguard. That way it tries to sell first any category and then dumps (just for food/herbs/junk it can't really sell, but still needed to be checked)

@mrhoribu mrhoribu marked this pull request as draft January 31, 2026 16:56
@mrhoribu mrhoribu marked this pull request as ready for review January 31, 2026 16:56
Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

Important

Looks good to me! 👍

Reviewed everything up to fe6a965 in 10 seconds. Click for details.
  • Reviewed 297 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 0 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.

Workflow ID: wflow_nT2saXWczgXt3NV2

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

force saving of migrated settings for overflow upon migration
@mrhoribu mrhoribu marked this pull request as draft February 1, 2026 01:11
@mrhoribu mrhoribu marked this pull request as ready for review February 1, 2026 01:11
Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

Important

Looks good to me! 👍

Reviewed everything up to bdb422b in 9 seconds. Click for details.
  • Reviewed 391 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 0 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.

Workflow ID: wflow_zcx6zmo34aUkKYgN

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

@Deysh Deysh mentioned this pull request Feb 1, 2026
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