Skip to content

Next

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
    • Help
    • Submit feedback
    • Contribute to GitLab
    • Switch to GitLab Next
  • Sign in / Register
Minds Frontend
Minds Frontend
  • Project
    • Project
    • Details
    • Activity
    • Releases
    • Dependency List
    • Cycle Analytics
    • Insights
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Charts
    • Locked Files
  • Issues 812
    • Issues 812
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge Requests 47
    • Merge Requests 47
  • 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 FrontendMinds Frontend
  • Merge Requests
  • !426

Open
Opened 1 week ago by Brian Hatchet@brianhatchet:speech_balloon:
  • Report abuse
Report abuse

(Feat) disable comments 526

Edited 1 week ago by Brian Hatchet

Check out, review, and merge locally

Step 1. Fetch and check out the branch for this merge request

git fetch origin
git checkout -b feat/disable-comments-526 origin/feat/disable-comments-526

Step 2. Review the changes locally

Step 3. Merge the branch and fix any conflicts that come up

git fetch origin
git checkout origin/master
git merge --no-ff feat/disable-comments-526

Step 4. Push the result of the merge to GitLab

git push origin master

Note that pushing to GitLab requires write access to this repository.

Tip: You can also checkout merge requests locally by following these guidelines.

Request to merge feat/disable-comments-526 into master
The source branch is 17 commits behind the target branch
Open in Web IDE
  • Email patches
  • Plain diff
Pipeline #71359013 passed for 3ebd1846 on feat/disable-comments-526
          Requires 2 more approvals from Devs and Deployers. Approved by
          Marcelo Rivera
          Marcelo Rivera
          Ben Hayward
          Ben Hayward
          Martin Santangelo
          Martin Santangelo
          Mark Harding
          Mark Harding
          Emiliano Balbuena
          Emiliano Balbuena
          Ready to be merged automatically. Ask someone with write access to this repository to merge this request

          Deletes source branch

          • Discussion 21
          • Commits 25
          • Pipelines 9
          • Changes 25
          7/9 threads resolved
          • Loading...
          • Brian Hatchet :speech_balloon: @brianhatchet changed milestone to %sprint: Interesting Iguana 1 week ago

            changed milestone to %sprint: Interesting Iguana

          • Brian Hatchet :speech_balloon: @brianhatchet changed title from Feat/disable comments 526 to (Feat) disable comments 526 1 week ago

            changed title from Feat/disable comments 526 to (Feat) disable comments 526

          • Brian Hatchet :speech_balloon: @brianhatchet added 4 commits 1 week ago

            added 4 commits

            • b8f503ae...d7f9cc7d - 3 commits from branch master
            • e93b4129 - Merge remote-tracking branch 'upstream/master' into feat/disable-comments-526

            Compare with previous version

          • Brian Hatchet :speech_balloon: @brianhatchet added 1 commit 1 week ago

            added 1 commit

            • 032dc9ae - fixing comment allowComment access leve

            Compare with previous version

          • Ben Hayward @benhayward.ben approved this merge request 1 week ago

            approved this merge request

          • Mark Harding @markeharding added MR::Awaiting Review scoped label 1 week ago

            added MR::Awaiting Review scoped label

          • Brian Hatchet :speech_balloon: @brianhatchet added 8 commits 1 week ago

            added 8 commits

            • 032dc9ae...12a0ab82 - 6 commits from branch master
            • b3689ba5 - Merge branch 'master' of gitlab.com:minds/front
            • 4b51976e - Merge branch 'master' into feat/disable-comments-526

            Compare with previous version

          • Brian Hatchet :speech_balloon: @brianhatchet added 1 commit 1 week ago

            added 1 commit

            • 95ca222a - Adding 'allow-comments-toggle' feature. Setting featuresservice assumption to false

            Compare with previous version

          • Brian Hatchet :speech_balloon: @brianhatchet added 45 commits 1 week ago

            added 45 commits

            • 95ca222a...bda69088 - 44 commits from branch master
            • 48216571 - Merge remote-tracking branch 'origin/master' into feat/disable-comments-526

            Compare with previous version

          • Brian Hatchet :speech_balloon: @brianhatchet added 12 commits 6 days ago

            added 12 commits

            • 48216571...3fad9777 - 11 commits from branch master
            • 5c29a195 - Merge remote-tracking branch 'upstream/master' into feat/disable-comments-526

            Compare with previous version

          • Mark Harding
            Mark Harding @markeharding started a thread on an old version of the diff 6 days ago
            Resolved by Brian Hatchet 5 days ago
          • Mark Harding
            Mark Harding @markeharding started a thread on the diff 6 days ago
            Last updated by Brian Hatchet 5 days ago
            src/app/modules/comments/comment/comment.component.ts
            87 94 private overlayModal: OverlayModalService,
            88 95 private cd: ChangeDetectorRef,
            89 96 private timeDiffService: TimeDiffService,
            90 private el: ElementRef
            97 private el: ElementRef,
            98 protected activityService: ActivityService
            91 99 ) {}
            92 100
            93 101 ngOnInit() {
            94 102 this.commentAge$ = this.timeDiffService.source.pipe(map(secondsElapsed => {
            95 103 return (this.comment.time_created - secondsElapsed * 0.01) * 1000;
            96 104 }));
            105 this.canReply = this.entity['allow_comments'];
            • Mark Harding
              Mark Harding @markeharding · 6 days ago
              Owner

              q. should this be instantiated from the tree/thread level?

            • Emiliano Balbuena
              Emiliano Balbuena @edgebal · 5 days ago
              Developer

              Adhering to Mark's question here. Remember that, if you don't want to be passing the property through the whole tree, you can also use a shared just-for-state @Injectable() service using providers: [] on the top level component (the comments list container), and injecting it on the constructor() part of the children components.

            • Brian Hatchet
              Brian Hatchet :speech_balloon: @brianhatchet · 5 days ago
              Maintainer

              I am confused. I am pretty sure that is exactly what I am doing.

              Provided here. https://gitlab.com/minds/front/blob/feat/disable-comments-526/src/app/modules/comments/list/list.component.ts#L28

              Then I just inject it into whatever component needs to respond at the constructor level.

            • Please register or sign in to reply
          • Mark Harding
            Mark Harding @markeharding started a thread on the diff 6 days ago
            Resolved by Brian Hatchet 2 days ago
          • Mark Harding
            Mark Harding @markeharding started a thread on the diff 6 days ago
            Resolved by Brian Hatchet 2 days ago
          • Emiliano Balbuena
            Emiliano Balbuena @edgebal started a thread on the diff 5 days ago
            Resolved by Brian Hatchet 2 days ago
          • Emiliano Balbuena
            Emiliano Balbuena @edgebal started a thread on the diff 5 days ago
            Resolved by Brian Hatchet 2 days ago
          • Emiliano Balbuena
            Emiliano Balbuena @edgebal started a thread on the diff 5 days ago
            Resolved by Brian Hatchet 2 days ago
          • Emiliano Balbuena
            Emiliano Balbuena @edgebal started a thread on the diff 5 days ago
            Resolved by Brian Hatchet 2 days ago
          • Emiliano Balbuena
            Emiliano Balbuena @edgebal started a thread on the diff 5 days ago
            src/app/modules/blogs/view/view.ts
            40 43
            41 44 scroll_listener;
            42 45
            43 menuOptions: Array<string> = ['edit', 'follow', 'feature', 'delete', 'report', 'subscribe', 'set-explicit', 'remove-explicit', 'rating'];
            46 menuOptions: Array<string> = ['edit', 'follow', 'feature',
            47 'delete', 'report', 'subscribe',
            48 'set-explicit', 'remove-explicit', 'rating',
            49 'allow-comments', 'disable-comments'];
            • Emiliano Balbuena
              Emiliano Balbuena @edgebal · 5 days ago
              Developer

              Check my comment above re: toggles using the same option ID.

            • Please register or sign in to reply
          • Brian Hatchet :speech_balloon: @brianhatchet added 13 commits 5 days ago

            added 13 commits

            • 5c29a195...87b682c6 - 12 commits from branch master
            • 7093e8d1 - Merge remote-tracking branch 'origin/master' into feat/disable-comments-526

            Compare with previous version

          • Brian Hatchet :speech_balloon: @brianhatchet added 1 commit 5 days ago

            added 1 commit

            • 3ebd1846 - CR changes

            Compare with previous version

          • Mark Harding @markeharding added Squad::Green scoped label 4 days ago

            added Squad::Green scoped label

          • Marcelo Rivera @eiennohi approved this merge request 1 day ago

            approved this merge request

          • 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
          Assignee
          Brian Hatchet's avatar Brian Hatchet @brianhatchet
          Assign to
          sprint: Interesting Iguana
          Milestone
          sprint: Interesting Iguana
          Assign milestone
          None
          Time tracking
          No estimate or time spent
          2
          Labels
          MR::Awaiting Review Squad::Green
          Assign labels
          • View project labels
          Lock merge request
          Unlocked
          6
          6 participants
          user avatar
          Ben Hayward
          user avatar
          Martin Santangelo
          user avatar
          Mark Harding
          user avatar
          Emiliano Balbuena
          user avatar
          Brian Hatchet
          user avatar
          Marcelo Rivera
          Reference: minds/front!426