Skip to content

Removing the last peer in a group does not remove the group from draft reports #12

@bbenligiray

Description

@bbenligiray
  1. Refer to a populated group in a report draft (and it works fine)
  2. Remove all peers from the group (which effectively removes the group)
  3. Sync the reports and it will error because a report draft is referring to a group that doesn't exist (anymore)

One way to deal with this is to parse all report drafts whenever the last peer is being removed from a group and remove it wherever it's used. However, removing the last peer from a group is not explicit enough of a signal that the group will no longer be used, the user may add another peer to the group right after removing its last member. Updating drafts automatically will then cause reports to be mishandled, which is quite risky. So the current behavior feels preferable.

Metadata

Metadata

Assignees

No one assigned

    Labels

    wontfixThis will not be worked on

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions