Skip to content

Next

  • Projects
  • Groups
  • Snippets
  • Help
  • Sign in / Register
Minds Backend - Engine
Minds Backend - Engine
  • Project
  • Repository
  • Issues 293
  • Merge Requests 40
  • CI / CD
  • Security & Compliance
  • Packages
  • Wiki
  • Snippets
  • Members
  • Collapse sidebar
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • Minds
  • Minds Backend - EngineMinds Backend - Engine
  • Merge Requests
  • !365

Merged
Opened 2 weeks ago by Marcelo Rivera@eiennohi
Report abuse

(fix): don't do anything if the entity isn't an image or video

closes #925 (closed)

Request to merge fix/sentry-issue-925 into master
Pipeline #88921424 passed with warnings for e3bac422 on fix/sentry-issue-925
          Merge request approved. Approved by
          Mark Harding
          Mark Harding
          Emiliano Balbuena
          Emiliano Balbuena
          Martin Santangelo
          Martin Santangelo
          Guy Thouret
          Guy Thouret
          Olivia Madrid
          Olivia Madrid
          Brian Hatchet
          Brian Hatchet

          Merged by Mark Harding 1 hour ago

          The changes were merged into master with bfac17ad

          The source branch has been deleted

          Closed #925 (closed)

          Pipeline #93460759 waiting for manual action for bfac17ad on master
                        • Discussion 4
                        • Commits 1
                        • Pipelines 2
                        • Changes 1
                        0/1 thread resolved
                        • Marcelo Rivera @eiennohi changed milestone to %Misc. Q4 2019 2 weeks ago

                          changed milestone to %Misc. Q4 2019

                        • Marcelo Rivera @eiennohi added Sprint::10/09 - Pink Panther Squad::Yellow scoped labels 2 weeks ago

                          added scoped labels

                        • Mark Harding @markeharding added MR::Requires Changes scoped label 2 weeks ago

                          added scoped label

                        • Mark Harding
                          Mark Harding @markeharding · 2 weeks ago
                          Owner

                          Linting issues @eiennohi

                        • Mark Harding
                          Mark Harding @markeharding started a thread on an old version of the diff 2 weeks ago
                          Last updated by Marcelo Rivera 2 weeks ago
                          Controllers/api/v1/media.php
                          36 36 return Factory::response(['status' => 'error']);
                          37 37 }
                          38 38
                          39 if (!in_array(['object:video', 'object:image'], $entity->type)) {
                          • Mark Harding
                            Mark Harding @markeharding · 2 weeks ago
                            Owner

                            type will always be object?

                          • Marcelo Rivera @eiennohi changed this line in version 2 of the diff 2 weeks ago

                            changed this line in version 2 of the diff

                          • Marcelo Rivera
                            Marcelo Rivera @eiennohi · 2 weeks ago
                            Developer

                            Just fixed this line. I think the issue came from someone going into /media/:guid, with an activity guid instead of an image or video. That's why I'm doing the check

                          • Please register or sign in to reply
                        • Marcelo Rivera @eiennohi added 1 commit 2 weeks ago

                          added 1 commit

                          • e3bac422 - (fix): don't do anything if the entity isn't an image or video

                          Compare with previous version

                        • Marcelo Rivera
                          Marcelo Rivera @eiennohi · 2 weeks ago
                          Developer

                          @markeharding, I just updated

                        • Mark Harding @markeharding approved this merge request 1 hour ago

                          approved this merge request

                        • Mark Harding @markeharding mentioned in commit bfac17ad 1 hour ago

                          mentioned in commit bfac17ad

                        • Mark Harding @markeharding merged 1 hour ago

                          merged

                        Please register or sign in to reply
                        0 Assignees
                        None
                        Misc. Q4 2019
                        Milestone
                        Misc. Q4 2019
                        Time tracking
                        No estimate or time spent
                        3
                        Labels
                        MR::Requires Changes Sprint::10/09 - Pink Panther Squad::Yellow
                        Lock merge request
                        Unlocked
                        10
                        10 participants
                        user avatar
                        Emiliano Balbuena
                        user avatar
                        Martin Santangelo
                        user avatar
                        Guy Thouret
                        user avatar
                        Olivia Madrid
                        user avatar
                        Brian Hatchet
                        user avatar
                        Rami Albatal
                        user avatar
                        Ben Hayward
                        Reference: minds/engine!365