Skip to content

  • Projects
  • Groups
  • Snippets
  • Help
  • Sign in / Register
Minds Frontend
Minds Frontend
  • Project overview
  • Repository
  • Issues 806
  • Merge Requests 54
  • 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
  • !657

Open
Opened 3 days ago by Ben Hayward@benhayward.ben
Report abuse

[Sprint/RollingRabbit](fix): Blocked channels eternally loading. #2244

Closes #2244

Summary

Currently, users on production who have nobody blocked will see an ever-spinning loading icon.

Testing

  • Check /settings/blocked-channels
  • Observe that when channels are blocked, they are shown, when channels are not blocked, the loading icon does not endlessly display.

Reasoning

I'm throwing an error to end the observable lifecycle. The filter with this approach is redundant.

Request to merge fix/blocked-channels-bug-2244 into master
The source branch is 8 commits behind the target branch
Open in Web IDE
Pipeline #95999897 passed with warnings for 12a67e6f on fix/blocked-channels-bug-2244
              Requires 3 more approvals from Devs, Deployers, and QA.
              Mark Harding
              Mark Harding
              Emiliano Balbuena
              Emiliano Balbuena
              Brian Hatchet
              Brian Hatchet
              Xander Miller
              Xander Miller
              Ready to be merged automatically. Ask someone with write access to this repository to merge this request

              Closes #2244

              Deletes source branch

              • Discussion 1
              • Commits 1
              • Pipelines 1
              • Changes 1
              0/1 thread resolved
              • Ben Hayward @benhayward.ben added MR::Awaiting Review Squad::Yellow scoped labels 3 days ago

                added scoped labels

              • Mark Harding
                Mark Harding @markeharding started a thread on the diff 3 minutes ago
                src/app/modules/settings/blocked-channels/blocked-channels.component.ts
                35 36 ngOnInit() {
                36 37 this.load(true);
                37 38 this.channels = this.blockListService.blocked.pipe(
                38 tap(() => {
                39 tap(list => {
                39 40 this.inProgress = true;
                40 41 this.moreData = false; // Support pagination in the future
                42
                43 if (list.length < 1) {
                44 throw new Error('No more entities');
                • Mark Harding
                  Mark Harding @markeharding · 3 minutes ago
                  Owner

                  "Channels" not entities.

                • Please register or sign in to reply
              • Mark Harding @markeharding added MR::Requires Changes scoped label and automatically removed MR::Awaiting Review label 3 minutes ago

                added scoped label and automatically removed label

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