FEATURE: multiple use invite links (PR #9813)

This commit allows Admins to create an invite link that can be used multiple times.

Creating Invite Link:

Screen Shot 2020-05-18 at 10 26 11

Invite Link listing page:

Screen Shot 2020-05-18 at 10 26 18

Invite Link redemption page:

Screen Shot 2020-05-18 at 12 09 47

Invite “Redeemed” tab showing all the accepted invites:

Screen Shot 2020-05-18 at 12 10 51

GitHub

This pull request introduces 4 alerts when merging bdd46e1288b7a1c0dc4284122fe15a8fc4cc73f1 into 862773ec83756a2deb6beaabbcb16756a98595f9 - view on LGTM.com

new alerts:

  • 4 for Unused variable, import, function or class

This pull request introduces 2 alerts when merging aa5856d542c839cc362983dc278cdecd7465c855 into 862773ec83756a2deb6beaabbcb16756a98595f9 - view on LGTM.com

new alerts:

  • 2 for Unused variable, import, function or class

missing a space here, will fail linting

would prefer readOnly instead of all the alias here

should be set there’s only one property

        .then(result => {

do we need a get here ?

        .catch(e => {

Why do you need a native array here ?

Also I would prefer this to be null, and the array initiated in init()

you can replace all of this by using or()

          .then(result => {
    ).then(invites => this.set("model", invites));

do we need the get here ? if currentUser is null having a get won’t save you from an error

maybe we should have a constant for “links” here

        confirm => {
        ).then(invite_model => {

can you try to remove this inline style while you are at it please ?

do we need email=null here ? why not just removed the email property

everything else is on one line and this if is done on 3 ? :stuck_out_tongue: