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 805
    • Issues 805
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge Requests 49
    • Merge Requests 49
  • 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
  • !425

Merged
Opened 1 week ago by Juan Manuel Solaro@juanmsolaro
  • Report abuse
Report abuse

[Sprint/InterestingIguana] (feat) ability to disable toaster notifications

Closes engine#239 (closed). Depends on engine!256 (merged). Also Closes #1450 (closed)

Edited 6 days ago by Juan Manuel Solaro
Request to merge feat/ability-to-disable-toaster-239 into master
  • Email patches
  • Plain diff
Pipeline #70714350 passed for e830d8ef on feat/ability-to-disable-toaster-239
          Merge request approved. Approved by
          Mark Harding
          Mark Harding
          Martin Santangelo
          Martin Santangelo
          Marcelo Rivera
          Marcelo Rivera
          Emiliano Balbuena
          Emiliano Balbuena
          Ben Hayward
          Ben Hayward
          Brian Hatchet
          Brian Hatchet

          Merged by Mark Harding 8 minutes ago

          The changes were merged into master with 12ba7805

          The source branch has been deleted

          Closed #1450 (closed) and engine#239 (closed)

          Pipeline #71428565 running for 12ba7805 on master
                • Discussion 5
                • Commits 4
                • Pipelines 4
                • Changes 9
                0/2 threads resolved
                • Loading...
                • Juan Manuel Solaro @juanmsolaro added 1 commit 1 week ago

                  added 1 commit

                  • 3c4b4d9f - (feat) support tags pipe on media page titles

                  Compare with previous version

                • Juan Manuel Solaro @juanmsolaro added 1 commit 1 week ago

                  added 1 commit

                  • 51d6b96c - (feat) add translation tags to locale files

                  Compare with previous version

                • Juan Manuel Solaro @juanmsolaro unmarked as a Work In Progress 1 week ago

                  unmarked as a Work In Progress

                • Juan Manuel Solaro @juanmsolaro changed the description 1 week ago

                  changed the description

                • Juan Manuel Solaro @juanmsolaro mentioned in merge request engine!256 (merged) 1 week ago

                  mentioned in merge request engine!256 (merged)

                • Brian Hatchet :speech_balloon: @brianhatchet approved this merge request 1 week ago

                  approved this merge request

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

                  approved this merge request

                • Juan Manuel Solaro @juanmsolaro changed the description 6 days ago

                  changed the description

                • Mark Harding
                  Mark Harding @markeharding started a thread on an old version of the diff 4 days ago
                  Last updated by Juan Manuel Solaro 4 days ago
                  src/app/modules/settings/general/general.component.html
                  98 98 </div>
                  99 99 <!-- END: MATURE CONTENT -->
                  100 100
                  101 <!-- START: TOASTER NOTIFICATIONS -->
                  102 <div class="m-settings--section m-border m-settings--toaster">
                  103 <h4 i18n="@@MINDS__SETTINGS__TOASTER_TITLE">Notifications</h4>
                  104 <div class="mdl-card__supporting-text">
                  105 <input type="checkbox"
                  106 id="flag_toaster"
                  107 (click)="change()"
                  108 name="toaster_notifications"
                  109 [(ngModel)]="toaster_notifications"
                  110 />
                  111 <label for="flag_toaster">
                  112 <ng-container i18n="@@MINDS__SETTINGS__TOASTER_LABEL">Enable or disable toaster notifications</ng-container>
                  • Mark Harding
                    Mark Harding @markeharding · 4 days ago
                    Owner

                    Can we rename to 'Enable toaster notifications'?

                  • Juan Manuel Solaro
                    Juan Manuel Solaro @juanmsolaro · 4 days ago
                    Developer

                    Totally

                  • Juan Manuel Solaro @juanmsolaro changed this line in version 4 of the diff 4 days ago

                    changed this line in version 4 of the diff

                  • Please register or sign in to reply
                  • Mark Harding
                    Mark Harding @markeharding · 4 days ago
                    Owner

                    The checkbox is always checked despite clicking on it and saving.

                  • Collapse replies
                  • Juan Manuel Solaro
                    Juan Manuel Solaro @juanmsolaro · 4 days ago
                    Developer

                    looking at this now

                  • Please register or sign in to reply
                • Mark Harding @markeharding added MR::Requires Changes scoped label 4 days ago

                  added MR::Requires Changes scoped label

                • Juan Manuel Solaro @juanmsolaro added 1 commit 4 days ago

                  added 1 commit

                  • e830d8ef - (fix) change checkbox label and corrected state

                  Compare with previous version

                • Juan Manuel Solaro @juanmsolaro added MR::Awaiting Review scoped label and automatically removed MR::Requires Changes label 4 days ago

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

                • Guy Thouret
                  Guy Thouret @gthouret · 4 days ago
                  Developer

                  Went to do my duty and test some MRs, but found I can no longer approve.

                  Setting toggles and persists.

                  Edited by Guy Thouret 4 days ago
                • Mark Harding @markeharding added Squad::Yellow scoped label 1 hour ago

                  added Squad::Yellow scoped label

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

                  approved this merge request

                • Mark Harding @markeharding merged 8 minutes ago

                  merged

                • Mark Harding @markeharding mentioned in commit 12ba7805 8 minutes ago

                  mentioned in commit 12ba7805

                • 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
                2
                Labels
                MR::Awaiting Review Squad::Yellow
                Assign labels
                • View project labels
                Lock merge request
                Unlocked
                8
                8 participants
                user avatar
                Martin Santangelo
                user avatar
                Marcelo Rivera
                user avatar
                Emiliano Balbuena
                user avatar
                Ben Hayward
                user avatar
                Brian Hatchet
                user avatar
                Mark Harding
                user avatar
                Guy Thouret
                Reference: minds/front!425

                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.