Skip to content
Snippets Groups Projects

Add global time tracking report

All threads resolved!

What does this MR do and why?

This is the first iteration about introducing the global time tracking report and, as such, some functionalities are definitely missing. Notable examples are:

  • filters do not apply the URL
  • cannot filter by project or group
  • filters don't use the filtered search bar
  • total spent time is missing
  • grouping issuables is missing
  • and more

Screenshots or screen recordings

Registrazione_schermo_2023-03-10_alle_19.37.15

Large view Small view
Schermata_2023-03-15_alle_12.49.33 Schermata_2023-03-15_alle_12.50.03

How to set up and validate locally

Optional: seed some timelog using this command

bundle exec rake db:seed_fu FILTER=37_timelogs SEED_TIMELOGS=true
  • Enable the global_time_tracking_report feature flag
    • gdk rails c then, from the rails console, Feature.enable(:global_time_tracking_report)
  • Add some time entries to one (or more) issuables (issues or MRs)
  • Go to http://127.0.0.1:3000/-/timelogs to view the report

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #344002

Edited by Marco Zille

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Marco Zille added 1 commit

    added 1 commit

    • 844992b6 - Applied suggestions from review

    Compare with previous version

  • removed review request for @pgascouvaillancourt

  • Marco Zille mentioned in issue #403819

    mentioned in issue #403819

  • Nick Leonard removed review request for @nickleonard

    removed review request for @nickleonard

  • Marco Zille changed the description

    changed the description

  • Nick Leonard approved this merge request

    approved this merge request

  • Marco Zille added 1 commit

    added 1 commit

    • 956b1f3d - Applied suggestions from review

    Compare with previous version

  • requested review from @pgascouvaillancourt

  • Marco Zille added 1 commit

    added 1 commit

    • 5af8b64e - Applied suggestions that slipped previously

    Compare with previous version

  • Niklas van Schrick
  • Paul Gascou-Vaillancourt requested review from @jannik_lehmann and removed review request for @pgascouvaillancourt

    requested review from @jannik_lehmann and removed review request for @pgascouvaillancourt

  • Paul Gascou-Vaillancourt approved this merge request

    approved this merge request

  • Marco Zille added 1 commit

    added 1 commit

    • b2bfff6b - Applied suggestions from review

    Compare with previous version

  • Marco Zille added 1 commit

    added 1 commit

    • dfc0d950 - Scoped translations to time tracking report

    Compare with previous version

  • Marco Zille added 1 commit

    added 1 commit

    Compare with previous version

  • Tetiana Chupryna
  • Tetiana Chupryna
  • Jannik Lehmann
  • Jannik Lehmann
  • mentioned in issue #404735 (closed)

  • requested review from @msedlakjakubowski

  • Marco Zille mentioned in issue #404745

    mentioned in issue #404745

  • Marco Zille added 1 commit

    added 1 commit

    • 7df96743 - Applied suggestions from review

    Compare with previous version

  • Marco Zille requested review from @phillipwells and removed review request for @msedlakjakubowski

    requested review from @phillipwells and removed review request for @msedlakjakubowski

  • Jannik Lehmann approved this merge request

    approved this merge request

  • Jannik Lehmann removed review request for @jannik_lehmann

    removed review request for @jannik_lehmann

  • Tetiana Chupryna approved this merge request

    approved this merge request

  • Marco Zille mentioned in issue #404790

    mentioned in issue #404790

  • Phillip Wells approved this merge request

    approved this merge request

  • Phillip Wells removed review request for @phillipwells

    removed review request for @phillipwells

  • Lee Tickett resolved all threads

    resolved all threads

  • merged

  • @zillemarco, how was your code review experience with this merge request? Please tell us how we can continue to iterate and improve:

    1. React with a :thumbsup: or a :thumbsdown: on this comment to describe your experience.
    2. Create a new comment starting with @gitlab-bot feedback below, and leave any additional feedback you have for us in the comment.

    Interested in learning more tips and tricks to solve your next challenge faster? Subscribe to the GitLab Community Newsletter for contributor-focused content and opportunities to level up.

    Thanks for your help! :heart:

    This message was generated automatically. You're welcome to improve it.

  • Lee Tickett mentioned in commit 0d017d93

    mentioned in commit 0d017d93

  • Lee Tickett resolved all threads

    resolved all threads

  • added workflowstaging label and removed workflowcanary label

  • Lee Tickett mentioned in epic &8429

    mentioned in epic &8429

    • Resolved by Lee Tickett

      I enabled the feature flag for me and you @zillemarco

      When I hit the new endpoint i'm getting a 500 from the GraphQL request at the moment.

      Digging in elastic I can see:

      Integer out of bounds: 2409876072 @ timelogs.totalSpentTime (TimelogConnection.totalSpentTime). Consider using ID or GraphQL::Types::BigInt instead.

      I'm going to log some time on this MR then run for today and see what happens :fingers_crossed:

  • Lee Tickett added 10m of time spent

    added 10m of time spent

  • Lee Tickett added 5m of time spent at 2023-04-09

    added 5m of time spent at 2023-04-09

  • Lee Tickett resolved all threads

    resolved all threads

  • mentioned in issue #406548 (closed)

  • Lee Tickett resolved all threads

    resolved all threads

  • Lee Tickett mentioned in issue #406549

    mentioned in issue #406549

  • Lee Tickett resolved all threads

    resolved all threads

  • Lee Tickett mentioned in issue #406550

    mentioned in issue #406550

  • Lee Tickett mentioned in issue #344002

    mentioned in issue #344002

  • Lee Tickett resolved all threads

    resolved all threads

  • mentioned in issue #407539 (closed)

  • Marco Zille mentioned in merge request !119124 (closed)

    mentioned in merge request !119124 (closed)

  • mentioned in merge request !132020 (merged)

  • Donald Cook mentioned in issue #502683

    mentioned in issue #502683

  • Please register or sign in to reply
    Loading