Skip to content

Next

  • Projects
  • Groups
  • Snippets
  • Help
  • Sign in / Register
Minds Frontend
Minds Frontend
  • Project overview
  • Repository
  • Issues 808
  • Merge Requests 55
  • CI / CD
  • Security & Compliance
  • Packages
  • Wiki
  • Snippets
  • Members
  • Collapse sidebar
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • Minds
  • Minds FrontendMinds Frontend
  • Merge Requests
  • !617

Merged
Opened 1 week ago by Ben Hayward@benhayward.ben
Report abuse

[Sprint/QuietQuail](feat) Rich embed open in media modals #2162

Summary

Opening youtube videos in modals.

Testing

This has the potential to affect

My test plan (note gifs aren't working at present on sandboxes): Visit: https://feat-rich-embed-modals-2162.minds.io/richembedtest/

  • Click through the content.
  • Test in groups, comments and discovery>latest.
  • Flip the feature flag via helm (any help, let me know).
  • Refresh and check that everything still works

Try a variety of content.

Edited 5 days ago by Ben Hayward
Request to merge feat/rich-embed-modals-2162 into master
Pipeline #93924102 running for d109f28f on feat/rich-embed-modals-2162
              Merge request approved. Approved by
              Mark Harding
              Mark Harding
              Emiliano Balbuena
              Emiliano Balbuena
              Brian Hatchet
              Brian Hatchet
              Xander Miller
              Xander Miller

              Merged by Mark Harding 1 minute ago

              The changes were merged into master with 524d294a

              The source branch has been deleted

              Mentions #2162

              Pipeline #94318908 running for 524d294a on master
                            • Discussion 7
                            • Commits 17
                            • Pipelines 9
                            • Changes 8
                            2/2 threads resolved
                            • Ben Hayward @benhayward.ben added MR::Requires Changes Squad::Yellow scoped labels 1 week ago

                              added scoped labels

                            • Ben Hayward @benhayward.ben changed title from WIP: Sprint/QuietQuail Rich embed open in media modals 2162 to WIP: Sprint/QuietQuail Rich embed open in media modals #2162 1 week ago

                              changed title from WIP: Rich embed open in media modals 2162 to WIP: Rich embed open in media modals 2162

                            • Ben Hayward @benhayward.ben added 7 commits 1 week ago

                              added 7 commits

                              • 7b38de6c...80069206 - 2 commits from branch master
                              • 9c6591b4 - got it in 3 clicks
                              • 74af9382 - openModal used
                              • 059b2411 - 2 clicks
                              • 16abac59 - Merge branch 'master' of gitlab.com:minds/front into feat/rich-embed-modals-2162
                              • 137b7820 - Style updates needed

                              Compare with previous version

                              Toggle commit list
                            • Ben Hayward @benhayward.ben added 2 commits 1 week ago

                              added 2 commits

                              • 617ee55b - Don't show overlay
                              • 9ffde279 - Tidied up

                              Compare with previous version

                            • Ben Hayward @benhayward.ben added 1 commit 1 week ago

                              added 1 commit

                              • 12dc5478 - Putting things behind feat flags and tidying

                              Compare with previous version

                            • Ben Hayward @benhayward.ben added 3 commits 1 week ago

                              added 3 commits

                              • 3ffcecb7 - Left in a comment
                              • cff8a4bd - Routing working
                              • 9283d879 - Got rid of grey bg on comment embed body and tidied.

                              Compare with previous version

                            • Ben Hayward
                              Ben Hayward @benhayward.ben started a thread on an old version of the diff 1 week ago
                              Resolved by Ben Hayward 6 days ago
                            • Ben Hayward @benhayward.ben added 1 commit 6 days ago

                              added 1 commit

                              • e103e3e3 - Testing and fixes

                              Compare with previous version

                            • Ben Hayward @benhayward.ben resolved all threads 6 days ago

                              resolved all threads

                            • Ben Hayward @benhayward.ben added 1 commit 6 days ago

                              added 1 commit

                              • f9431bac - Get rid of vimeo autoplay

                              Compare with previous version

                            • Ben Hayward @benhayward.ben changed the description 6 days ago

                              changed the description

                            • Ben Hayward @benhayward.ben added 2 commits 6 days ago

                              added 2 commits

                              • 790ed6da - Removed feature flag changes.
                              • 74aeb47e - E2E testing

                              Compare with previous version

                            • Ben Hayward @benhayward.ben unmarked as a Work In Progress 5 days ago

                              unmarked as a Work In Progress

                            • Ben Hayward @benhayward.ben added MR::Awaiting Review scoped label and automatically removed MR::Requires Changes label 5 days ago

                              added scoped label and automatically removed label

                            • Ben Hayward @benhayward.ben mentioned in issue #2165 (closed) 3 days ago

                              mentioned in issue #2165 (closed)

                            • Mark Harding
                              Mark Harding @markeharding started a thread on the diff 3 days ago
                              Resolved by Ben Hayward 1 day ago
                            • Mark Harding @markeharding added MR::Requires Changes scoped label and automatically removed MR::Awaiting Review label 3 days ago

                              added scoped label and automatically removed label

                            • Mark Harding
                              Mark Harding @markeharding · 3 days ago
                              Owner

                              Looking good so far. The description/title looks to be overflowing (potentially just remove this?).

                              Screenshot_2019-11-04_at_09.27.50 https://feat-rich-embed-modals-2162.minds.io/newsfeed/1038021148513669138

                            • Ben Hayward @benhayward.ben added 1 commit 1 day ago

                              added 1 commit

                              • d109f28f - Took out description and title

                              Compare with previous version

                            • Ben Hayward @benhayward.ben resolved all threads 1 day ago

                              resolved all threads

                            • Ben Hayward @benhayward.ben added MR::Awaiting Review scoped label and automatically removed MR::Requires Changes label 12 hours ago

                              added scoped label and automatically removed label

                            • Mark Harding
                              Mark Harding @markeharding · 16 minutes ago
                              Owner

                              This is also no longer working for me at https://feat-rich-embed-modals-2162.minds.io/newsfeed/1038021148513669138

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

                              added scoped label and automatically removed label

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

                              Ok looks like something is up with sandbox site not having the modal feature flag set.

                            • Mark Harding @markeharding added MR::Ready to Merge scoped label and automatically removed MR::Requires Changes label 2 minutes ago

                              added scoped label and automatically removed label

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

                              approved this merge request

                            • Ben Hayward
                              Ben Hayward @benhayward.ben · 1 minute ago
                              Developer

                              Oh nice, thank you very much.

                            • Mark Harding @markeharding merged 1 minute ago

                              merged

                            • Mark Harding @markeharding mentioned in commit 524d294a 1 minute ago

                              mentioned in commit 524d294a

                            Please register or sign in to reply
                            0 Assignees
                            None
                            None
                            Milestone
                            None
                            Time tracking
                            No estimate or time spent
                            2
                            Labels
                            MR::Ready to Merge Squad::Yellow
                            Lock merge request
                            Unlocked
                            10
                            10 participants
                            user avatar
                            Emiliano Balbuena
                            user avatar
                            Marcelo Rivera
                            user avatar
                            Guy Thouret
                            user avatar
                            Martin Santangelo
                            user avatar
                            Brian Hatchet
                            user avatar
                            Olivia Madrid
                            user avatar
                            Juan Manuel Solaro
                            Reference: minds/front!617