Skip to content
Snippets Groups Projects

[Sprint/ModestMonkey](feat): Added in pending boost limit for #591

Closed Ben requested to merge feat/pending-boost-limit-591 into master
All threads resolved!

Helm values helm-charts!9 (closed) Closes #591 (closed)

To test

The simplest way is going to be to log in as admin, then open 2 windows.

image

Next, make posts numbered 1, 2, 3 and 4 on the newsfeed, and boost them.

On 4, you should receive an error indicating that you have exceeded the pending boost limit.

After this, go to the other window (admin panel), refresh, and approve a boost. Try the boost 4 again, it should let you.

Try approve all, and do 3 fresh boosts.

Edited by Ben

Merge request reports

Checking pipeline status.

Requires 1 approval from Deployers.

Closed by BenBen 5 years ago (Oct 14, 2019 8:26pm UTC)

Merge details

  • The changes were not merged into master.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Ben added StatusRequires Changes label and removed 1 deleted label

    added StatusRequires Changes label and removed 1 deleted label

  • Ben resolved all threads

    resolved all threads

  • Ben changed the description

    changed the description

  • Ben added 1 deleted label and removed StatusRequires Changes label

    added 1 deleted label and removed StatusRequires Changes label

  • Ben added 1 commit

    added 1 commit

    • 03e2d8ad - updated variable name to better reflect value held

    Compare with previous version

  • Ben changed title from Added in pending boost limit for #591 (closed) to {+Sprint/ModestMonkey: +}Added in pending boost limit for #591 (closed)

    changed title from Added in pending boost limit for #591 (closed) to {+Sprint/ModestMonkey: +}Added in pending boost limit for #591 (closed)

  • Admin team report no need for these currently, can be re-opened if the need arises.

  • closed

  • Please register or sign in to reply
    Loading