You’re viewing Paradaux’s public workspaceSign in to vote and comment Get taskl.ie for your team

Paradaux
Issues
PAR-127

PAR-127

Search in paradaux, or jump to…

Actions

Change themeDark, light or system↵

Copy current URL

Navigation

Issues

Board

Projects

Deployments

Teams

↑ ↓ navigate↵ openEsc close

Loading issues

Built with taskl.ie

Issues Projects Search

Loading issue

PAR-127 · /business reload should actually re-read config values… · Paradaux · taskl.ie

Workspace

Projects Deployments

Teams

PParadauxSign in

PAR-127

/business reload should actually re-read config values (not just raw config)

The existing /business reload calls message.reload() + business.reloadConfig() but the config POJOs (FirmConfiguration, BalanceTaxConfiguration) read their values once in their constructors into final fields, so services keep stale values. Give those POJOs a reload() that re-reads from plugin.getConfig(), make their fields non-final, and call them from ReloadCommand so firm.owned-limit/create-cooldown and the balance-tax brackets actually update live.

Properties

Status: Done
Priority: No priority
Assignee: Unassigned
Labels: No labels
Project: Business
Milestone: No milestone

Details

Votes: 0 votes
Created: Jun 7

Sign in

Sign in to edit this issue, vote and comment.

Activity

tesks created the issue Jun 7
ParadauxIO linked a commit: Commit e54ae61 — Make /business reload actually refresh config values (PAR-127) Jun 7

Sign in to comment

Join the conversation, react to comments and vote on this issue.