Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[VO-1064 & VO-1067] feat: Take into account feedback on announcements #2184

Merged
merged 3 commits into from
Sep 18, 2024
Merged

[VO-1064 & VO-1067] feat: Take into account feedback on announcements #2184

merged 3 commits into from
Sep 18, 2024

Commits on Sep 13, 2024

  1. feat: Only user action mark a announcement seen 

    Previously, announcements was considered to have been seen when it appeared. Now, it's when the user decides to move on to the next announcement that the one displayed is considered to have been seen.
    @cballevre
    cballevre committed Sep 13, 2024
    Configuration menu
    Copy the full SHA
    57bab7c View commit details
    Browse the repository at this point in the history

  2. feat: Limit to 5 the number of announcement shown 

    Before the API was limiting the number of response. This limit has been removed so that users can go 5 by 5 through the announcement they haven't seen yet.
    @cballevre
    cballevre committed Sep 13, 2024
    Configuration menu
    Copy the full SHA
    c028087 View commit details
    Browse the repository at this point in the history

  3. feat: Show announcement 1h after the feature has been activated 

    To avoid spamming the interface with new users, we're putting a delay between the first opening of the Home and the appearance of the announcements dialog.
    @cballevre
    cballevre committed Sep 13, 2024
    Configuration menu
    Copy the full SHA
    2d4948b View commit details
    Browse the repository at this point in the history