* premium:
  Factor out BuyNumber
  Inject price
  Bill after charge
  Inline useless methods
  Remove unused method
  Fix test
  charge for premium nums
  refactor registration_test to use real Tn's
  two changes related to registration forms
  price functionality on Tns
  Remove legacy card charge behaviour
  two fixes related to eSIM nicks:
  Factor out reload customer, check balance, maybe bill
  Unify more invite code handling
  Remove mostly-unused final_message
  Remove legacy card charge behaviour

Since that's what we actually care about.

Also makes JustCharge more generic for future use

instead of this blind retry

And use a better transaction id

And move price method where it actually goes

- new buy_number.rb form for premium nums
- change mail.rb to accept a @price

0 unless LocalInventory, in which case price is fetched from postgres

We used to have them add card in web then charge after, but these days
we charge them in web for this first tx to get 3DS, so this code path is
all but dead and we can remove it.

1. actually expose nickanem at eSIM order-time
2. pass nickname to `SIMOrder.commit` as non-kw arg in eSIM sublcass

1. actually expose nickanem at eSIM order-time
2. pass nickname to `SIMOrder.commit` as non-kw arg in eSIM sublcass

Using the optional field on Activation or the InviteCode flow now use
the same code and do the same things. Always check balance and proceed
to bill if we've used a parent code, otherwise never do that.

Fixes the subaccount-from-onboarding guard as well.

Was only being used for PayPal transitions (which are mostly done) and
only when transitioning to BTC (which they usually don't) and only
provided extra reinforcement of a message they've already seen by now.
Let's not complicate the code for that.

We used to have them add card in web then charge after, but these days
we charge them in web for this first tx to get 3DS, so this code path is
all but dead and we can remove it.

name can be set at registration and bulk edited later as well

Instead of raw sql issue

* '388-warn-about-top-up-limits' of https://git.sr.ht/~amolith/sgx-jmp:
  feat(form): conditionally display max top-up amount
  test(credit-card-sale): add missing openstruct require
  style: wrap YARD comment lines
  refactor: remove duplicated logic from BuyAccountCreditForm
  refactor: set DeclinedError defaults to nil
  fix: raise and check for errors correctly
  refactor: protect max_declines
  fix: redo amount validation in the form
  docs: add Yard documentation to code we touched
  feat: add upper limit to top-ups based on trust level

Remove range from the form, but pass max_top_up_amount through and only display
it if the user isn't tombed.

Not entirely sure what was changed, but the test started failing until I added
the missing require.

Replace incorrect `CreditCardSale::TooLowError` and
`CreditCardSale::TooHighError` with correct `AmountTooLowError` and
`AmountTooHighError` respectively.

Document methods and attributes in `BuyAccountCreditForm`, `CreditCardSale`
error classes, and the `TrustLevel` module subclasses.

* bulk-order-api:
  New bulk order API

instead of Bandwidth.move'ing it

invalid: adj., 1) not registered