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 845
    • Issues 845
    • 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
  • Issues
  • #1594

Closed
Open
Opened 1 week ago by Minds Gitbot@mindsgitbot:bug:
  • Report abuse
  • New issue
Report abuse New issue

(bug): Notifications for deleted boosts break the notification window

Summary:

'Boost' notifcations usually carry a link to the post. When this post doesn't exist, the notification still comes through. This results in all the notifications prior being erased, making the new notifications unresponsive and rendering the rest of the website unresponsive.

The notifications below the unassigned boost notification become empty, and the website becomes seemingly unresponsive, whatever you click on won't open.

The bug seems to be because of the unassigned 'Boost' notification. Because this specific notification is only visible in the 'all notifcation' tab, the other tabs (votes, subscribers, comments etc.) are not affected. When the unassigned 'Boost' notification gets burried by other notifications, the site will not break unless you load it by scrolling down to it.

To explain the bug in simple terms: When you boost a post, you can delete that post. The boost notification will point at a deleted post, find nothing and break the website through the notification system. Because of the way minds caches posts, a deleted post can and will still be processed. After deleting the post you made a timebomb that will break your notification system.

Steps to reproduce:

  1. Create a media post on your profile (image)

  2. Boost the post on your profile (must be of type Newsfeed)

  3. Delete the post permanently on your profile

  4. Revoke the boosted post in the Boost Console (this will also happen if boost approved or rejected).

  5. Check the notification window, it should behave incorrectly and show critical damage to the website

Browser and OS

Mozilla Firefox 68.0 reported by: @aobachan

What is the current bug behavior?

Screenshot should be self explanatory.

What is the expected correct behavior?

If a notification is somehow corrupted and causing an issue, the front end should simply be able to disregard it, though the underlying cause here needs to be investigated and amended.

Relevant logs and/or screenshots

image

Possible fixes

(If you can, link to the line of code that might be responsible for the problem)

/label ~"T - Bug" ~"S - Triage:new"

Please solve the reCAPTCHA

We want to be sure it is you, please confirm you are not a robot.

Edited 1 week ago by Ben Hayward

Related issues
0

    Related merge requests
    1
    • Opened 11 minutes ago
      Jul 28, 2019 6:36pm GMT+0000
      [Sprint/JollyJellyfish](bug): Fixed issue causing notifications to mem leak out and break
      Opened 11 minutes ago
      Jul 28, 2019 6:36pm GMT+0000
      !465
    When this merge request is accepted, this issue will be closed automatically.
    • Ben Hayward @benhayward.ben added Triage::New Type::Bug (Triage) scoped labels 1 week ago

      added Triage::New Type::Bug (Triage) scoped labels

    • Ben Hayward @benhayward.ben made the issue visible to everyone 1 week ago

      made the issue visible to everyone

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

      Unclear on step 4, messaged @medworthy who brought this to my attention.

    • aoba
      aoba @aobars · 1 week ago

      Step 4 is achievable in three ways You either

      1. Revoke the boost in the boost console
      2. Wait for the boost to be accepted
      3. Wait for the boost to be denied

      Revoking is the easiest way, because you can complete that step immediately. Revoking it should send you a notification containing the text [You revoked your boost Your offer has been refunded.] You must delete the post before you revoke the boost.Knipsel

      Edited by aoba 1 week ago
    • Ben Hayward @benhayward.ben changed the description 2 times within 10 minutes 1 week ago

      changed the description 2 times within 10 minutes

    • Ben Hayward @benhayward.ben added Priority::1 - High Product::Boost Type::Bug scoped labels and automatically removed Triage::Questions Type::Bug (Triage) labels 1 week ago

      added Priority::1 - High Product::Boost Type::Bug scoped labels and automatically removed Triage::Questions Type::Bug (Triage) labels

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

      Thank you very much for bringing this to our attention @aobars, I've now reproduced it, and escalated it to bug status.

    • Ben Hayward @benhayward.ben changed title from (site-breaking bug) Unassigned 'Boost' Notifications breaks the notification window and the whole website to (bug): Notifications for deleted boosts break the notification window 1 week ago

      changed title from (site-breaking bug) Unassigned 'Boost' Notifications breaks the notification window and the whole website to (bug): Notifications for deleted boosts break the notification window

    • Ben Hayward @benhayward.ben assigned to @benhayward.ben 4 days ago

      assigned to @benhayward.ben

    • Ben Hayward @benhayward.ben changed milestone to %sprint: Jolly Jellyfish 4 days ago

      changed milestone to %sprint: Jolly Jellyfish

    • Ben Hayward @benhayward.ben changed weight to 5 4 days ago

      changed weight to 5

    • Ben Hayward @benhayward.ben added Squad::Green scoped label 3 days ago

      added Squad::Green scoped label

    • Ben Hayward @benhayward.ben added Status::InProgress scoped label 3 days ago

      added Status::InProgress scoped label

    • Ben Hayward @benhayward.ben automatically removed Status::InProgress label 2 days ago

      automatically removed Status::InProgress label

    • Ben Hayward @benhayward.ben added Status::InProgress scoped label 2 days ago

      added Status::InProgress scoped label

    • Ben Hayward @benhayward.ben added Status::Review scoped label and automatically removed Status::InProgress label 15 minutes ago

      added Status::Review scoped label and automatically removed Status::InProgress label

    • Ben Hayward @benhayward.ben changed milestone to %sprint: Jolly Jellyfish 15 minutes ago

      changed milestone to %sprint: Jolly Jellyfish

    • Ben Hayward @benhayward.ben changed weight to 5 15 minutes ago

      changed weight to 5

    • Ben Hayward @benhayward.ben moved from minds#669 (closed) 15 minutes ago

      moved from minds#669 (closed)

    • Ben Hayward @benhayward.ben mentioned in merge request !465 11 minutes ago

      mentioned in merge request !465

    • 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
    Ben Hayward's avatar Ben Hayward @benhayward.ben
    Assign to
    None
    Epic
    None
    sprint: Jolly Jellyfish
    Milestone
    sprint: Jolly Jellyfish
    Assign milestone
    None
    Time tracking
    No estimate or time spent
    None
    Due date
    None
    5
    Labels
    Priority::1 - High Product::Boost Squad::Green Status::Review Type::Bug
    Assign labels
    • View project labels
    5
    Weight
    5
    Confidentiality
    Not confidential
    Lock issue
    Unlocked
    5
    5 participants
    user avatar
    Ben Hayward
    user avatar
    Mark Edworthy
    user avatar
    aoba
    user avatar
    Minds Gitbot
    user avatar
    Luculent
    Reference: minds/front#1594