Meeds-io / MIPs

The Meeds Improvement Proposal repository
0 stars 0 forks source link

Stream Readability #124

Closed srenault-meeds closed 3 weeks ago

srenault-meeds commented 3 months ago

Rationale

More knowledge is shared in the stream, more it can be difficult to know what is useful for me as a user. Indeed:

  1. Posts that I have already read are still displayed in their whole content
  2. Comment with links can be really noisy for eyes (link length + link embedded thumbnail) and it is not useful for any reader.

1. Functional Requirements

Top User Stories

  1. Read/unread activity display

Current As a user, when new activity is posted

New display / UX

  1. Links readability / posting

Current As a user, when adding multiple links to a post Then only the first is embedded And both links are fully displayed

New display / UX

  1. Links readability / commenting Current As a user, when adding links in a comment Then links can be embedded And links are fully displayed

New display / UX When adding links to a comment Then:

  1. Links display When adding links Then an icon before the link is added so we can quickly see what this is about
    • internal links
    • external links
    • if not know, a link icon is displayed

Impacts

Gamification

NA

Notifications

NA

Analytics

NA

Unified Search

NA

2. Technical Requirements

Expected Volume & Performance

Security

Extensibility

Configurability

Upgradability

Existing Features

Feature Flags

Other Non Functional Requirements

3. Impacts

Documentation

Training

4. Software Architecture

Services & processing

Neither Rest nor WebSockets APIs will be added here.

The existing webSocket ActivityStreamWebSocketService will be used to broadcast activities events . Thus we need to subscribe to the Channel “/eXo/Application/ActivityStream” and listen to the following activities and comments actions:

To check the read/unread activity, we will adopt the same approach as the unread badge display. Therefore, we will utilize the activity.metadata.unread object to determine whether the activity should be displayed in its full or collapsed form.

If the activity is fully displayed, we need to reload the page in order to collapse the activity content.

We Have to add a new param collapsed (default = true) to the activity content extension to enable the display of full or collapsed activity content.

srenault-meeds commented 3 months ago

To check @margondicco thanks

margondicco commented 3 months ago

Go Fonc (+ we should keep embeded video in comment)

srenault-meeds commented 3 months ago

Ok. Thanks (added the embedded video case)

SaraBoutej commented 3 months ago

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

rdenarie commented 3 months ago

ok for me

SaraBoutej commented 3 weeks ago

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

margondicco commented 3 weeks ago

Hello thank you @SaraBoutej . Tested and validated

cc @srenault-meeds

rdenarie commented 3 weeks ago

Ok for me, you can proceed to merge