search

Meeds-io / MIPs

The Meeds Improvement Proposal repository
0 stars 0 forks source link

Overview review #104

Closed srenault-meeds closed 7 months ago

srenault-meeds commented 7 months ago

Rationale

After a year of using the overview page, UX needs to be slightly improved:

So we suggest a slight review of this page to answer these feedbacks

1. Functional Requirements

Top User Stories

Overview items are reversed so we have in first row, the information of what to do:

image

When accessing the platform and nothing has been setup yet Then placeholders are reviewed to be more concise and with better balance between them

image

For weekly achievements portlet, then display the pie-chart as is with no subtitles as currently

image

AND when clicking to see all, then display the achievement list of the user in the drawer

image

When no kudos yet, the same placeholder is displayed in the same place no matter we have badges or not

image

And the contrary as well (kudos but no badges)

image

And an access to dedicated drawer is proposed for each (kudos creation or badges)

image

When no wallet yet, then no need to display the perkstore placeholder

image

When wallet init, but no perks, then depending on ability to create, two placeholders are proposed

PRECISION

Condidering that the overview lists users, we need to make sure that username is not displayed when not logged in. Thus, the URL for avatar of users will be modified to hide the username with an anonymous ID

Impacts

Gamification

Overview gadgets review

Notifications

NA

Analytics

NA

Unified Search

NA

2. Technical Requirements

Upgradability

The Overview page will changed to reorder the portlets. This change will be applied automatically since the import mode is merge of meeds and global sites.

Feature Flags

No feature flags

3. Software Architecture

Access

The UIs has to be enhanced without adding new portlets. (UI modifications only for better UX)

srenault-meeds commented 7 months ago

@margondicco can you please check this? Thanks

margondicco commented 7 months ago

Hello Go Fonc Thanks

srenault-meeds commented 7 months ago

Thanks. @boubaker ready for tech spec

boubaker commented 7 months ago

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

rdenarie commented 7 months ago

Go tech

srenault-meeds commented 7 months ago

FYI I have added this to avoid any security breach:

PRECISION

Condidering that the overview lists users, we need to make sure that username is not displayed when not logged in. Thus, the URL for avatar of users will be modified to hide the username with an anonymous ID

boubaker commented 7 months ago
  1. ACCs ready to be reviewed by DAO members (eXo: @rdenarie @margondicco ) 2.PRs to develop ready for review by DAO members (eXo: @rdenarie)
boubaker commented 7 months ago
  1. ACCs ready to be reviewed by DAO members (eXo: @rdenarie @margondicco )
  2. PRs to develop ready for review by DAO members (eXo: @rdenarie)

Should I help by a video if it's difficult for the review ?

rdenarie commented 7 months ago

All pr validated exect meeds one

boubaker commented 7 months ago

All pr validated exect meeds one

Answered, in fact the import mode flags, when defined in meeds.war will affect Meeds Package only.

rdenarie commented 7 months ago

Ok. For the portal meeds, we are not impacted. Should we do something for our overview page (https://community.exoplatform.com/portal/engage/overview) ?

For group navigation : why do you reload all group navigation ? Should we do something on our side ? What if we do not overwrite theses navigation on exo side ? Can you confirm that space group navigations are not impacted ?

boubaker commented 7 months ago

For group navigation : why do you reload all group navigation ?

We need to delete the navigations that has been deleted by code as well (No UI to do it)

For group navigation : ... Should we do something on our side ?

For group navigations no, because the MIP #88 will delete them anyway and will replace them by a site.

[!IMPORTANT] For public site, yes, you will need to reimport the pages using the flag io.meeds.public.portalConfig.metadata.importmode=OVERWRITE only once (Delete flag once restarted) + re-publish manually on Tribe the public site once server restarted.

For group navigation : ... What if we do not overwrite theses navigation on exo side ?

No need to them anymore since the group navigations aren't defined in group sites anymore with MIP #88

[!IMPORTANT] What will remain to reimport from sources on your side, will be the overview/Participate page. All of this will be enhanced (for pages upgrade) after fixing the Enhancement issue https://github.com/Meeds-io/meeds/issues/1307

boubaker commented 7 months ago

I let you review the PR again if this answers your questions.

rdenarie commented 7 months ago

al lpr validated

rdenarie commented 7 months ago

Can you ping me when all merged (for rebase) ?

boubaker commented 7 months ago

al lpr validated

Thanks, all PRs merged