Meeds-io / MIPs

The Meeds Improvement Proposal repository
0 stars 0 forks source link

Pending Status #122

Closed srenault-meeds closed 1 month ago

srenault-meeds commented 4 months ago

Rationale

More you have contributors, more you need to organize the contribution review. Currently, any action submitted is considered accepted. The problem is that:

In addition to this, we have observed that some contributions captured automatically need to be checked afterwards. Indeed, for example, a user can add a twitter account with no follower and he can Like, RT a tweet. Result: this twitter account has no follower so it is inefficient for our community

1. Functional Requirements

Top User Stories

Regarding account connection: When someone connects his account, then the action for connecting this account is pending by default AND any related action to this account connection can be manually added as locked until this action app connection is accepted rejecting the achievement doesn't disconnect the account

Regarding automatic actions No change, any contribution is by default accepted If needed, the acceptation can be changed to rejected or to pending no notification flow identified for this case

Regarding manual actions Any manual action is, by design, pending The owner when creating the action cannot change this parameter for now

Review of contributions

image
  1. Adding a comment will be done directly from the stream

Regarding rewards computing

To study

Impacts

Gamification

Connectors and profile list Contributions board to review Contributions are added once accepted

Notifications

Add a notification type for contribution status: to provide a way to inform when accepted/rejected

Analytics

Unified Search

2. Technical Requirements

Expected Volume & Performance

Security

NA

Extensibility

Add new types of activity actions to update the status of the contribution from the stream.

Configurability

NA

Upgradability

NA

Existing Features

The existing achievements Board has to be improved.

Feature Flags

No feature flag is needed.

Property Name Default Value Target Audience Functional Behaviour

4. Software Architecture

Security

NA

Services & processing

The default PENDING value of automatic realization status will be known from the trigger config (default false). For a manual rule, always true.

A new notification type is to be added GamificationActionVerifiedNotification.

Update the earned points account based on the pending status.

Data and persistence

We will have to add in Database Table GAMIFICATION_ACTIONS_HISTORY a new field REWARDED of type BOOLEAN to hold the realization rewarding status(default FALSE).

Migration strategy

no need for migrating old realization.

srenault-meeds commented 4 months ago

To check please @margondicco @plamarque

plamarque commented 4 months ago

How about the CANCELLED and DELETED statuses ?

srenault-meeds commented 4 months ago

Currently it is considered as "rejected" or it is empty. To avoid too much actions, I suggest to consider it as "rejected" with no ability to accept it, nor review it. WDYT?

plamarque commented 4 months ago

I suggest to consider it as "rejected" with no ability to accept it, nor review it.

OK, these states are final anyway. And anything particular to consider for notifications when reaching these statuses?

srenault-meeds commented 4 months ago

Good question. If we notify program owners when a contribution is submitted, then why not notifying them when the contribution is cancelled/rejected?

We have two options:

  1. Notification -> Too much noise for nothing
  2. No notification but a page with a placeholder -> Less noise

WDYT?

srenault-meeds commented 4 months ago

FYI, we have reviewed the part regarding the notification when a contribution is submitted:

New requirement

The following will be worked later

srenault-meeds commented 4 months ago

Added screenshots and reviewed the notification part. Ready for go-fonc check @margondicco

margondicco commented 3 months ago

Go Fonc

AzmiTouil commented 3 months ago

Ready for tech review by DAO Members (eXo: @rdenarie )

rdenarie commented 3 months ago

Go tech

AzmiTouil commented 1 month ago

MIP ready for review by the DAO members (cc @rdenarie ). Acc up & running.

rdenarie commented 1 month ago

Ok for me on technical side

margondicco commented 1 month ago

ok for me on functional side

boubaker commented 1 month ago

Merged to develop