set_global_multiplier event does not include old value

[ritaifeoluwa]

Description

The FarmingPool contract's set_global_multiplier function does not emit an event with the old multiplier value. The event only includes the new value.

Current behavior

env.events().publish(
    (symbol_short!("boost"), symbol_short!("mult_set")),
    multiplier,
);

Expected behavior

Include old and new:

let old_multiplier = read_global_multiplier(&env);
// ...
env.events().publish(
    (symbol_short!("boost"), symbol_short!("mult_set")),
    (old_multiplier, multiplier),
);

Why this matters

Off-chain indexers need both values for audit trails and rollback scenarios.

Labels

enhancement, farming-pool, good first issue

[payfoxX]

@payfoxX has applied to work on this issue as part of the Stellar Wave Program's 8th wave.

I love to fix this issue, kindly assign it to me. Thanks

ℹ️ Repo Maintainers: To accept this application, review their application or assign @payfoxX to this issue.

[Muyideen-js]

@Muyideen-js has applied to work on this issue as part of the Stellar Wave Program's 8th wave.

Hi, I can fix this

ℹ️ Repo Maintainers: To accept this application, review their application or assign @Muyideen-js to this issue.

[dollfins]

@dollfins has applied to work on this issue as part of the Stellar Wave Program's 8th wave.

• Goal: Write a natural 1-2 sentence application to work on a specific GitHub issue. * Repository: SmartDropLabs/smartdrop-contracts * Issue Title: set_global_multiplier event does not include old value * Issue Description: The set_global_multiplier function in FarmingPool emits an event with only the new multiplier. It should include both the old and new values for better indexing/auditing. * Constraints: * Natural tone. * 1-2 sentences. * Mention one concrete implementation or testing

ℹ️ Repo Maintainers: To accept this application, review their application or assign @dollfins to this issue.

[wrvnnull]

@wrvnnull has applied to work on this issue as part of the Stellar Wave Program's 8th wave.

I can fix this. I’ll capture the old multiplier before the update and include both old and new values in the set_global_multiplier event so off-chain indexers have the full state change for audit and rollback.

ℹ️ Repo Maintainers: To accept this application, review their application or assign @wrvnnull to this issue.

[VeronicDev]

@VeronicDev has applied to work on this issue as part of the Stellar Wave Program's 8th wave.

Hi, I can fix this pls assign

ℹ️ Repo Maintainers: To accept this application, review their application or assign @VeronicDev to this issue.

[bade22brazy]

@bade22brazy has applied to work on this issue as part of the Stellar Wave Program's 8th wave.

 hello maintainer, kindly assign these issues to me. i am sure of handling thenm after careful and thorough review

ℹ️ Repo Maintainers: To accept this application, review their application or assign @bade22brazy to this issue.

[drips-wave]

Congratulations, @bade22brazy! 🎉 Your application was accepted by the repo's maintainers, and the issue is due on August 31, 2026.

🧑‍💻 @bade22brazy: Please resolve the issue such that the repo's maintainers have enough time to review your contribution before the due date. You'll earn Points for completing the issue on-time, which will make you eligible for a share of the Stellar Wave Program's reward pool.

Warning

When opening a PR, please link it to this issue to ensure it gets tracked accurately. Points are awarded when this issue is marked as completed by the maintainer.

🤠 Repo maintainers: Please keep an eye on the contributor's progress and review their work before the due date. You can manage this issue, including adjusting its complexity and points, here.

🌊 Happy Wave 🌊