Skip to content

Next

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
    • Help
    • Submit feedback
    • Contribute to GitLab
    • Switch to GitLab Next
  • Sign in / Register
Minds Backend - Engine
Minds Backend - Engine
  • Project
    • Project
    • Details
    • Activity
    • Releases
    • Dependency List
    • Cycle Analytics
    • Insights
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Charts
    • Locked Files
  • Issues 167
    • Issues 167
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge Requests 43
    • Merge Requests 43
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
    • Charts
  • Packages
    • Packages
    • List
    • Container Registry
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • Minds
  • Minds Backend - EngineMinds Backend - Engine
  • Merge Requests
  • !262

Merged
Opened 1 week ago by Marcelo Rivera@eiennohi
  • Report abuse
Report abuse

[Sprint/InterestingIguana] (fix): resolve comments by urn

closes #457 (closed) depends on front!454

Edited 4 hours ago by Marcelo Rivera
Request to merge fix/resolve-comments-by-urn into master
  • Email patches
  • Plain diff
Pipeline #71230089 failed for a33cb825 on fix/resolve-comments-by-urn
          Merge request approved. Approved by
          Mark Harding
          Mark Harding
          Brian Hatchet
          Brian Hatchet
          Ben Hayward
          Ben Hayward
          Martin Santangelo
          Martin Santangelo
          Emiliano Balbuena
          Emiliano Balbuena
          Rami Albatal
          Rami Albatal

          Merged by Mark Harding 15 minutes ago

          The changes were merged into master with a72d6646

          The source branch has been deleted

          Closed #457 (closed)

          Pipeline #71318337 passed for a72d6646 on master
                  • Discussion 1
                  • Commits 4
                  • Pipelines 3
                  • Changes 7
                  1/1 thread resolved
                  • Loading...
                  • Brian Hatchet :speech_balloon: @brianhatchet approved this merge request 6 days ago

                    approved this merge request

                  • Mark Harding
                    Mark Harding @markeharding started a thread on an old version of the diff 6 days ago
                    Resolved by Marcelo Rivera 1 day ago
                  • Mark Harding @markeharding added MR::Requires Changes scoped label 6 days ago

                    added MR::Requires Changes scoped label

                  • Marcelo Rivera @eiennohi added 1 commit 3 days ago

                    added 1 commit

                    • aa29430f - (fix): add phpdoc and also support string urns

                    Compare with previous version

                  • Marcelo Rivera @eiennohi resolved all threads 1 day ago

                    resolved all threads

                  • Mark Harding @markeharding added MR::Awaiting Review scoped label and automatically removed MR::Requires Changes label 23 hours ago

                    added MR::Awaiting Review scoped label and automatically removed MR::Requires Changes label

                  • Mark Harding @markeharding added 1 commit 6 hours ago

                    added 1 commit

                    • a33cb825 - (fix): parent path must have colons

                    Compare with previous version

                  • Marcelo Rivera @eiennohi changed the description 4 hours ago

                    changed the description

                  • Marcelo Rivera @eiennohi mentioned in merge request front!454 4 hours ago

                    mentioned in merge request front!454

                  • Mark Harding @markeharding approved this merge request 16 minutes ago

                    approved this merge request

                  • Mark Harding @markeharding merged 16 minutes ago

                    merged

                  • Mark Harding @markeharding mentioned in commit a72d6646 15 minutes ago

                    mentioned in commit a72d6646

                  • You're only seeing other activity in the feed. To add a comment, switch to one of the following options.
                  Please register or sign in to reply
                  0 Assignees
                  None
                  Assign to
                  None
                  Milestone
                  None
                  Assign milestone
                  None
                  Time tracking
                  No estimate or time spent
                  1
                  Labels
                  MR::Awaiting Review
                  Assign labels
                  • View project labels
                  Lock merge request
                  Unlocked
                  7
                  7 participants
                  user avatar
                  Brian Hatchet
                  user avatar
                  Ben Hayward
                  user avatar
                  Martin Santangelo
                  user avatar
                  Emiliano Balbuena
                  user avatar
                  Rami Albatal
                  user avatar
                  Mark Harding
                  user avatar
                  Marcelo Rivera
                  Reference: minds/engine!262

                  Revert this merge request

                  This will create a new commit in order to revert the existing changes.

                  Switch branch
                  Cancel
                  A new branch will be created in your fork and a new merge request will be started.

                  Cherry-pick this merge request

                  Switch branch
                  Cancel
                  A new branch will be created in your fork and a new merge request will be started.