feat(faucet): wait tx inclusion

[GabrielMartinezRodriguez]

Linked Issues

• closes https://linear.app/happychain/issue/HAPPY-517/faucet-await-tx-inclusion-before-returning-from-the-service

Description

Faucet await tx inclusion before returning from the service and then invalidate the happy balance to refresh it

• Include all relevant context (but no need to repeat the issue's content).
• Draw attention to new, noteworthy & unintuitive elements.

Toggle Checklist

Checklist

Basics

• B1. I have applied the proper label & proper branch name (e.g. norswap/build-system-caching).
• B2. This PR is not so big that it should be split & addresses only one concern.
• B3. The PR targets the lowest branch it can (ideally master).

Reminder: PR review guidelines

Correctness

• C1. Builds and passes tests.
• C2. The code is properly parameterized & compatible with different environments (e.g. local,
  testnet, mainnet, standalone wallet, ...).
• C3. I have manually tested my changes & connected features.

< INDICATE BROWSER, DEMO APP & OTHER ENV DETAILS USED FOR TESTING HERE >

< INDICATE TESTED SCENARIOS (USER INTERFACE INTERACTION, CODE FLOWS) HERE >

• C4. I have performed a thorough self-review of my code after submitting the PR,
  and have updated the code & comments accordingly.

Architecture & Documentation

• D1. I made it easy to reason locally about the code, by (1) using proper abstraction boundaries,
  (2) commenting these boundaries correctly, (3) adding inline comments for context when needed.
• D2. All public-facing APIs are documented in the docs.
  Public APIS and meaningful (non-local) internal APIs are properly documented in code comments.
• D3. If appropriate, the general architecture of the code is documented in a code comment or
  in a Markdown document.
• D4. An appropriate Changeset has been generated (and committed) with make changeset for
  breaking and meaningful changes in packages (not required for cleanups & refactors).

[cloudflare-workers-and-pages]

Deploying happychain with    Cloudflare Pages

Latest commit:	8a3cf6b
Status:	⚡️  Build in progress...

View logs

[GabrielMartinezRodriguez]

• Faucet: Limit usage to once per time window, increase quantity, and return the remaining wait time in case of a rate limit error. #727
• feat(faucet): wait tx inclusion #720 👈 (View in Graphite)
• feat(txm): added txm callbacks and promises when tx status change #719
• master

This stack of pull requests is managed by Graphite. Learn more about stacking.

[linear]

HAPPY-517 Faucet: await tx inclusion before returning from the service

cf. #661 (comment)

I think the service should await inclusion of the tx, so we can simply await response from the server on the wallet and invalidate the token balance query when the faucet completes. This will lead to a smooth flow: click > sending... > token sents, balance is updated.