Skip to content

GitLab

  • Menu
Projects Groups Snippets
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • G gitlabhq1
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 21
    • Issues 21
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 12
    • Merge requests 12
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Packages & Registries
    • Packages & Registries
    • Package Registry
    • Infrastructure Registry
  • Analytics
    • Analytics
    • CI/CD
    • Repository
    • Value stream
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • gpt
  • large_projects
  • gitlabhq1
  • Merge requests
  • !6747

Merged
Created Apr 10, 2014 by Administrator@rootOwner

Expose event and mergerequest timestamps in API

  • Overview 7
  • Commits 1
  • Changes 2

Created by: mrts

What does this MR do?

  • Exposes event and mergerequest timestamps in API
  • Makes code more DRY by moving title, description, state and timestamps to base class ProjectEntity

Are there points in the code the reviewer needs to double check?

I have to apologize that I have no way to run the unit tests as I'm currently using a Windows machine, so I can only hope that tests don't break because of this.

However, we have been using this change for a long time in our GitLab server, so I'm quite sure it won't break any functionality.

Also, see https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/64

Why was this merge request needed?

This merge request enables API users to create a "Changes in project since yesterday" view by filtering events by timestamp (imagine e.g. a project dasboard with GitLab widget). More details at https://groups.google.com/forum/#!topic/gitlabhq/grWbJp8Kp78

Also, it makes Events and MergeRequests consistent with other API entities like Issue.

What are the relevant feature requests?

http://feedback.gitlab.com/forums/176466-general/suggestions/5210611-gitlab-api-expose-updated-at-and-created-at-on-ev

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: github/fork/mrts/expose-event-and-mergerequest-timestamps-in-api