Meeds-io / MIPs

The Meeds Improvement Proposal repository
0 stars 0 forks source link

Contribute Site Review #155

Closed srenault-meeds closed 2 months ago

srenault-meeds commented 3 months ago

Rationale

After few months using it, the Contribute Site needs to be reviewed for better understanding. Moreover, pages and portlets listed in this site needs to be updated for compliance with layout options (style framework, responsiveness of portlets depending on the height, etc.)

1. Functional Requirements

Top User Stories

Navigation Review

The contribute site navigation will be reviewed as follow:

Pages Review

  1. Image Portlet:

    • Ensure the width of the image is still displayed no matter the device used
    • Provide options for users to crop image depending on the zone where the image is added
    • Compliance with Fixed or Dynamic Section
  2. Display options for portlets to make sure the user can set it upon the expected layout

    • Remove the maximum of items to list for programs and actions
    • Propose for each portlet options to sort the list and to display a number of items Example: image
  1. Rework of badges display
    • Better look / Bigger badge image
  1. Review of Rewards, kudos portlet

    • Kudos: Rework the portlet to be compliant with new layout and style
    • Rewards: Rework the portlet to be compliant with new layout and style
    • Ability to see transactions and rewards activity image
  2. Contributions Board: only list the contributions of the current user. Keep the "hosted" view for program owners only in an other app

Impacts

Gamification

NA

Notifications

NA

Analytics

NA

Unified Search

NA

2. Technical Requirements

Expected Volume & Performance

N/A

Security

N/A

Extensibility

N/A

Configurability

N/A

Upgradability

Contribute Page has to be automatically upgraded since its layout is updated.

Existing Features

N/A

Feature Flags

N/A

Other Non Functional Requirements

N/A

3. Software Architecture

Security

N/A

Access

The UI of existing portlets will change to make the display dynamically updated using the ResizeObserver API. The limit of items to fetch to display has to be independent from height to not having to refetch results each time the blocks are resized. Thus, a maximum of items has to be retrieved by default, only if the whole block isn't displayed.

Services & processing

N/A

Data and persistence

N/A

boubaker commented 2 months ago

Ready for Func & Tech review by DAO members (cc @margondicco @rdenarie )

rdenarie commented 2 months ago

Ok for me

margondicco commented 2 months ago

@srenault-meeds ok for me

boubaker commented 2 months ago

PRs & ACCs ready for review (eXo: @margondicco @rdenarie )

srenault-meeds commented 2 months ago

Thnaks @boubaker I have added programs, actions and I have used apps and layout. LGTM.

I let @margondicco check.

I will prepare the documentation once ok for you

margondicco commented 2 months ago

Hello

the portlet My reputation is displayed in the app registry

image

IMO we must remove the portlet

boubaker commented 2 months ago

the portlet My reputation is displayed in the app registry

In fact, it's deleted for new servers, but for the servers which had deployed the 7.0.0-M07 (App Registry MIP), it has to be deleted manually. If not deleted and used in a page, the instance will display the Kudos Overview (An upgrade plugin has been already used to modify the instance reference).

srenault-meeds commented 2 months ago

hello @margondicco

Two documentation items have been added:

I let you confirm if ok to merge it

margondicco commented 2 months ago

Hello ok for me

boubaker commented 2 months ago

Hello ok for me

Thanks, @rdenarie WDYT ?

rdenarie commented 2 months ago

Ok for me, you can merge this MIP