Skip to content

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

Open
Opened 5 hours ago by Marcelo Rivera@eiennohi
Report abuse

Allow users to permanently hide "share" buttons in settings

  • Overview 3
  • Commits 4
  • Pipelines 3
  • Changes 9
0/3 threads resolved

closes #2462

depends on engine!463

Edited 5 hours ago by Marcelo Rivera
Request to merge feat/hide-share-buttons into master
Open in Web IDE
Pipeline #116938569 passed with warnings for 0fb8b7b6 on feat/hide-share-buttons
              Requires 3 more approvals from Devs, Deployers, and QA.
              Martin Santangelo Rami Albatal Brian Hatchet Emiliano Balbuena Xander Miller
              Ready to be merged automatically. Ask someone with write access to this repository to merge this request

              Closes #2462

              Deletes source branch

              • Marcelo Rivera @eiennohi changed the description 5 hours ago

                changed the description

                • Brian Hatchet
                  Brian Hatchet :speech_balloon: @brianhatchet · 6 minutes ago
                  Developer

                  Ok, finally got the review site working with the latest and greatest changes. Unfortunately, the share button disabling is not working

                  • Go to settings
                  • Uncheck share settings
                  • Save
                  • Reload settings, note how it's properly unchecked
                  • Go to the feed
                  • Click on the date of a single entity
                  • Share buttons are still there

                  image

                  Note my settings are confirmed image

                • Please register or sign in to reply
              • Brian Hatchet :speech_balloon: @brianhatchet added Community label 5 minutes ago

                added label

              • Brian Hatchet :speech_balloon: @brianhatchet added Status::Requires Changes scoped label 5 minutes ago

                added scoped label

              • Brian Hatchet :speech_balloon: @brianhatchet removed Community label 5 minutes ago

                removed label

              • Brian Hatchet
                Brian Hatchet :speech_balloon: @brianhatchet started a thread on the diff 3 minutes ago
                cypress/integration/settings/settings.spec.js
                265 cy.get('.m-btn--action').contains('Save').click().wait('@postSettings').then(xhr => {
                266 expect(xhr.status).to.equal(200);
                267 expect(xhr.response.body.status).to.equal('success');
                268 });
                269
                270 cy.wait(1000);
                271
                272 cy.visit('/blog/edit/new');
                273
                274 uploadAvatar();
                275 createBlogPost(title, body, true);
                276
                277 cy.get('m-social-icons').should('not.exist');
                278 deleteBlogPost();
                279 });
                280
                • Brian Hatchet
                  Brian Hatchet :speech_balloon: @brianhatchet · 3 minutes ago
                  Developer

                  Need a spec test to make sure the share bar goes away when this setting is turned off.

                • Please register or sign in to reply
              • Brian Hatchet
                Brian Hatchet :speech_balloon: @brianhatchet started a thread on the diff 2 minutes ago
                src/app/modules/media/view/view.component.html
                327 327 [url]="siteUrl + 'media/' + entity.guid"
                328 328 [title]="entity.title || ''"
                329 329 [embed]="entity"
                330 *ngIf="entity && !attachment.shouldBeBlurred(entity)"
                330 *ngIf="
                331 entity &&
                332 !attachment.shouldBeBlurred(entity) &&
                333 !entity.ownerObj?.hide_share_buttons
                • Brian Hatchet
                  Brian Hatchet :speech_balloon: @brianhatchet · 2 minutes ago
                  Developer

                  What if they are looking at someone else's entity. We need to check the logged in user's settings, not the entity

                • Please register or sign in to reply
              Please register or sign in to reply
              0 Assignees
              None
              None
              Milestone
              None
              Time tracking
              No estimate or time spent
              1
              Labels
              Status::Requires Changes
              Lock merge request
              Unlocked
              10
              10 participants
              user avatar
              user avatar
              user avatar
              user avatar
              user avatar
              user avatar
              user avatar
              Reference: minds/front!776