Skip to content
Snippets Groups Projects

POC: Add /remind_me quick action

Merged Kerri Miller requested to merge 20490-add-remind-quick-action into master

What does this MR do and why?

This proof-of-concept adds a new quick action /remind_me <date in future> that will create a Notification/ToDo at the scheduled time. It uses scheduled Sidekiq jobs to create a ToDo for the user. It was built in this manner as an exercise in a fast :iteration: experiment to gather feedback and interest before committing to a scheduled wider release.

Screenshots

Command hinting

image

Reminder set

image

ToDo

image

Related to #20490

Edited by Kerri Miller

Merge request reports

Merged results pipeline #1264804062 passed

Pipeline: Ruby 3.1 as-if-foss

#1264814423

    Pipeline: E2E GDK

    #1264820970

      Pipeline: GitLab

      #1264818275

        +2

        Merged results pipeline passed for b974f583

        Test coverage 81.40% (16.84%) from 2 jobs

        Merged by Tetiana ChuprynaTetiana Chupryna 11 months ago (Apr 23, 2024 11:33pm UTC)

        Loading

        Pipeline #1265223914 passed

        Pipeline passed for 913caba5 on master

        Test coverage 64.58% (16.84%) from 2 jobs
        10 environments impacted.

        Activity

        Filter activity
        • Approvals
        • Assignees & reviewers
        • Comments (from bots)
        • Comments (from users)
        • Commits & branches
        • Edits
        • Labels
        • Lock status
        • Mentions
        • Merge request status
        • Tracking
      • 3 Warnings
        :warning: 2abd4584: Commits that change 30 or more lines across at least 3 files should describe these changes in the commit body. For more information, take a look at our Commit message guidelines.
        :warning: d5e55a4f: The commit subject must contain at least 3 words. For more information, take a look at our Commit message guidelines.
        :warning: 82649d4a: The commit subject must contain at least 3 words. For more information, take a look at our Commit message guidelines.
        1 Message
        :book: CHANGELOG missing:

        If this merge request needs a changelog entry, add the Changelog trailer to the commit message you want to add to the changelog.

        If this merge request doesn't need a CHANGELOG entry, feel free to ignore this message.

        Reviewer roulette

        Category Reviewer Maintainer
        analytics instrumentation @bastirehm profile link current availability (UTC+2, 9 hours ahead of author) Maintainer review is optional for analytics instrumentation
        backend @ddieulivol profile link current availability (UTC+2, 9 hours ahead of author) @brytannia profile link current availability (UTC+2, 9 hours ahead of author)

        Please check reviewer's status!

        • available Reviewer is available!
        • unavailable Reviewer is unavailable!

        Please refer to documentation page for guidance on how you can benefit from the Reviewer Roulette, or use the GitLab Review Workload Dashboard to find other available reviewers.

        Sidekiq queue changes

        This merge request contains changes to Sidekiq queues. Please follow the documentation on changing a queue's urgency.

        These queues were added:

        • issuable_create_reminder

        If needed, you can retry the :repeat: danger-review job that generated this comment.

        Generated by :no_entry_sign: Danger

        Edited by Ghost User
      • Kerri Miller added 1 commit

        added 1 commit

        Compare with previous version

      • Kerri Miller resolved all threads

        resolved all threads

      • Kerri Miller added 159 commits

        added 159 commits

        Compare with previous version

      • Kerri Miller added 1 commit

        added 1 commit

        • 9e31787d - Move /remind_me to issue & MR actions

        Compare with previous version

      • Kerri Miller added 1 commit

        added 1 commit

        • 86a95b16 - Move /remind_me to issuables

        Compare with previous version

      • Kerri Miller added 1 commit

        added 1 commit

        • a0e9098b - Add metrics tracking definitions

        Compare with previous version

      • Kerri Miller added 5 commits

        added 5 commits

        • 0bd17f98 - Move /remind_me back to issue/MR file
        • 5f3cad42 - Pass class name to worker
        • 193bdffa - Add basic worker that creates todo against target
        • 8a6cbe7f - Update worker queues to include issuable_create_reminder
        • bfe28d8c - Update sidekiq_queues to include issuable_create_reminder

        Compare with previous version

      • Kerri Miller added 1 commit

        added 1 commit

        Compare with previous version

      • Kerri Miller marked this merge request as ready

        marked this merge request as ready

      • Kerri Miller changed the description

        changed the description

      • Loading
      • Loading
      • Loading
      • Loading
      • Loading
      • Loading
      • Loading
      • Loading
      • Loading
      • Loading
      • Please register or sign in to reply
        Loading