search

Meeds-io / MIPs

The Meeds Improvement Proposal repository
0 stars 0 forks source link

My Craft Site Review #156

Closed srenault-meeds closed 1 week ago

srenault-meeds commented 1 month 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

image

Stream page will be moved to this site. Precision:

And this stream page will be reworked a little:

Section (Dynamic 2 columns - 8+4)

Following portlets will be removed from stream page

Precision for spaces gadget:

  1. Gadget listing spaces with different settings proposed for the administrator:
    • List spaces the user is member of (set to true by default)
    • Spaces of the current user: 4 by default (from 0 to 25)
    • Spaces recently visited during a period: 2 by default (from 0 to 25) for 30 days by default (from 1 to 90 days)
    • Spaces most active during a period: 2 by default (from 0 to 25) for 30 days by default (from 1 to 90 days) Precisions: External users won't see recently visited and most active. Only their spaces are listed
  2. If the height is fixed and there are too much items to list, a scrollbar is proposed
  3. A see all allows the user to access the complete list, organized by type (spaces, recent, most active) in a drawer with load more if needed
  4. Header display, contextual depending on what to display
    • If two or more space type to list, then display "Spaces"
    • If only spaces of the current user, display "Your Spaces"
    • If only most active spaces, display "Popular Spaces"
    • If only recently visited, display "Recent Spaces"
  5. Item display Spaces
    • Space Avatar
    • Space Name (2 lines maximum)
    • In case, the space is hidden or has been deleted, then the space will be identified as "Hidden" and not accessible

Removal of instances

Other pages will be reviewed to adapt some cases

Precision: For existing platform, don't override the root page as some customers have customized it

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

No auto upgrade will be made for sites meeds and mycraft to not affect existing layouts.

Existing Features

N/A

Feature Flags

N/A

Other Non Functional Requirements

N/A

3. Software Architecture

Security

N/A

Access

This MIP will just make some UI adjustments and reconfigure the mycraft site pages and navigations. We will need, to add an additional Widget to develop in Social which will list the recent spaces. In addition, the Who's online widget has to be reworked to hide itself when an external user attempts to display it.

Services & processing

N/A

Data and persistence

N/A

boubaker commented 3 weeks ago

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

rdenarie commented 3 weeks ago

ok for me

srenault-meeds commented 2 weeks ago

@margondicco I let you check please.

Regarding the notes App, this is not sure we will provide a widget. Indeed, we know that you're working on notes currently and we don't want to create something that will need to be reviewed weeks after.

Thanks

margondicco commented 2 weeks ago

Hello

The MIP128 will be merged soon. If it's possible the notes widget could be a great improvement.

Some clients customize the site My craft. So the improvements must not removed the customization

For example :

Image

srenault-meeds commented 2 weeks ago

Ok requirement added regarding the root page.

margondicco commented 2 weeks ago

Thank you go fonc

srenault-meeds commented 2 weeks ago

Edit: Added detailed specification regarding notes gadget to make sure we are aligned about it

srenault-meeds commented 1 week ago

Edit: @margondicco I suggest you to review the spec as I have reviewed it following internal discussions. Thanks

margondicco commented 1 week ago

Hello re go fonc :-)

srenault-meeds commented 1 week ago

Thanks.

I let you check @boubaker if the tech spec needs to be reviewed

boubaker commented 1 week ago

Thanks.

I let you check @boubaker if the tech spec needs to be reviewed

Thanks, ready for tech spec review again (eXo: @rdenarie )

rdenarie commented 1 week ago

Ok for me

boubaker commented 1 week ago

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

BTW: I recommend to test MIPs Bundle on Tribe Dev in order to test the modifications on an existing installation. BTW2: I think an adaptation is needed for MyCraft Site definition overridden here https://github.com/exoplatform/digital-workplace/blob/develop/digital-workplace-webapps/src/main/webapp/WEB-INF/conf/sites/portal/mycraft/navigation.xml

rdenarie commented 1 week ago

Ok for me, you can proceed to merge

DW PR : https://github.com/exoplatform/digital-workplace/pull/142

srenault-meeds commented 3 days ago

Documentation added @Julien-Dubois-eXo @margondicco if needed

https://docs.meeds.io/meeds-guides/user-guide/exploring-a-meeds-hub/entering-your-workspace