Skip to content
Snippets Groups Projects

mint UD instead of deposit + disable burn

Merged Hugo Trentesaux requested to merge 279-check-if-ud-is-using-well-pallet-balances into master
1 unresolved thread

closes #279 (closed)
closes #284 (closed)
closes #231 (closed) (did not want to do a separate MR for this one)

Edited by Hugo Trentesaux

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
331 331 core::num::NonZeroU16::new(current_ud_index)
332 332 .expect("unreachable because current_ud_index is never zero."),
333 333 );
334 let _ = T::Currency::deposit(who, uds_total, Precision::Exact);
334 // Currency is issued here
335 let actual_total = T::Currency::mint_into(who, uds_total)?;
  • It's OK but you should accept !308 (merged) before.

  • added 1 commit

    Compare with previous version

  • Hugo Trentesaux requested review from @tuxmain and removed review request for @c-geek

    requested review from @tuxmain and removed review request for @c-geek

  • Pascal Engélibert approved this merge request

    approved this merge request

  • added 1 commit

    Compare with previous version

  • Hugo Trentesaux enabled an automatic merge when the pipeline for 1a2c9df3 succeeds

    enabled an automatic merge when the pipeline for 1a2c9df3 succeeds

  • mentioned in commit 2ab1e958

  • Please register or sign in to reply
    Loading