Skip to content
This repository was archived by the owner on Dec 2, 2024. It is now read-only.

PLT-646 Removed the TotalFunds effect of WalletEffect.#718

Merged
koslambrou merged 1 commit intomainfrom
PLT-646-remove-total-funds-effect-in-wallet-effect
Sep 22, 2022
Merged

PLT-646 Removed the TotalFunds effect of WalletEffect.#718
koslambrou merged 1 commit intomainfrom
PLT-646-remove-total-funds-effect-in-wallet-effect

Conversation

@koslambrou
Copy link
Contributor

In short, we won't be using the wallet to query blockchain information. This is the responsability of the indexing component. See ADR-5 for more details.

Pre-submit checklist:

  • Branch
    • Tests are provided (if possible)
    • Commit sequence broadly makes sense
    • Key commits have useful messages
    • Formatting, PNG optimization, etc. are updated
  • PR
    • Self-reviewed the diff
    • Useful pull request description
    • Reference the ADR in the PR and reference the PR in the ADR (if revelant)
    • Reviewer requested

@koslambrou koslambrou marked this pull request as ready for review September 21, 2022 19:04
@koslambrou koslambrou marked this pull request as draft September 21, 2022 19:04
@koslambrou koslambrou force-pushed the PLT-646-remove-total-funds-effect-in-wallet-effect branch from f035640 to b3d9dca Compare September 21, 2022 19:30
In short, we won't be using the wallet to query blockchain information.
This is the responsability of the indexing component.
See our ADRs for more details.
@koslambrou koslambrou force-pushed the PLT-646-remove-total-funds-effect-in-wallet-effect branch from b3d9dca to 1c0672c Compare September 21, 2022 19:35
@koslambrou koslambrou requested a review from a user September 21, 2022 21:11
@koslambrou koslambrou marked this pull request as ready for review September 21, 2022 21:11
@koslambrou koslambrou merged commit edf99a8 into main Sep 22, 2022
@koslambrou koslambrou deleted the PLT-646-remove-total-funds-effect-in-wallet-effect branch September 22, 2022 10:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant