Buy/Sell RAM Notifications and Logging

[DenisCarriere]

Change Description

Add buy RAM logging by including additional inline actions and notifications via the use of require_recipient.

bytes & quant are computed values based on Bancor algorithm market.

This allows payer or receiver to confirm exact bytes sent/received via notifications.

API Changes

• Add require_recipient(receiver) on buyram & buyrambytes actions

ACTION: logbuyram

/**
 * Logging for buyram & buyrambytes action
 *
 * @param payer - the ram buyer,
 * @param receiver - the ram receiver,
 * @param quantity - the quantity of tokens to buy ram with.
 * @param bytes - the quantity of ram to buy specified in bytes.
 * @param ram_bytes - the ram bytes held by receiver after the action.
 */
[[eosio::action]]
void logbuyram( const name& payer, const name& receiver, const asset& quantity, int64_t bytes, int64_t ram_bytes );

🚧 Feedback

• add logsellram ACTION
• ACTION return value of new_bytes

[DenisCarriere]

confirm logging using buy_bytes_out

[ericpassmore]

Changed to merge into branch ram_actions. Need to keep in sync with antlerIO/reference_contracts. That repo is also being updated with faster finality changes. Having the changes on a branch, and then merging in main will make it easier to manage these two separate workstreams.

[ericpassmore]

Approving, to merge into ram_actions branch. Test coverage covered in other PRs, and test coverage will be reviewed and verified before integration into main branch.

[ericpassmore]

Is it worth testing this? Pushing the top level action and then checking for the inline actions executed?

[ericpassmore]

Is it worth testing this? Pushing the top level action and then checking for the inline actions executed?