Skip to content

Conversation

@kfastov
Copy link
Contributor

@kfastov kfastov commented Sep 30, 2025

  • Add an owner-only emergency withdrawal flow that checks the token pause state via the TokenManagement component,
    transfers the full balance, and emits an EmergencyWithdrawal event.
  • Wrap the component’s pause/unpause inside pause_protocol_token / unpause_protocol_token so external callers use the
    existing pause machinery and events.
  • Extend the integration suite with success/negative cases around the emergency exit (balance assertions + event spy).

@kfastov kfastov marked this pull request as ready for review October 2, 2025 16:37
@kfastov
Copy link
Contributor Author

kfastov commented Oct 2, 2025

@Pvsaint Hi! I’m trying to understand where the “pause state” for this function should come from. On one hand, we have self.paused_functions, but it isn’t referenced anywhere else in the contract and I’m not sure which function key it should use. On the other hand, the components (e.g. token_management_component) expose pause()/unpause(), but those aren’t callable from the main contract today, because the component isn’t embedded nor are its methods forwarded.

I temporarily forwarded the methods, but that solution feels hacky. Could you clarify what the intended flow is supposed to be?

@Pvsaint Pvsaint self-requested a review October 2, 2025 21:01
@Pvsaint Pvsaint merged commit 37281ae into MetroLogic:main Oct 2, 2025
1 check passed
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