Skip to content

Fix deletion of multiple features#4235

Merged
Withalion merged 3 commits intomasterfrom
bugfix/fix-multiple-feature-deletion-bug
Dec 8, 2025
Merged

Fix deletion of multiple features#4235
Withalion merged 3 commits intomasterfrom
bugfix/fix-multiple-feature-deletion-bug

Conversation

@gabriel-bolbotina
Copy link
Copy Markdown
Contributor

@gabriel-bolbotina gabriel-bolbotina commented Nov 24, 2025

Fixed issue when deleting multiple features for the second time, as described in this comment:
#4097 (comment)

fixes #4097

@Withalion
Copy link
Copy Markdown
Contributor

Withalion commented Nov 24, 2025

let's wait for builds to finish until merging, and also probably for @IvaKuklica approval

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Nov 24, 2025

Pull Request Test Coverage Report for Build 19892288606

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 28 unchanged lines in 3 files lost coverage.
  • Overall coverage decreased (-0.1%) to 59.586%

Files with Coverage Reduction New Missed Lines %
mm/app/multieditmanager.cpp 6 50.4%
mm/core/merginuserauth.cpp 9 68.24%
mm/core/merginapi.cpp 13 75.15%
Totals Coverage Status
Change from base Build 19892247371: -0.1%
Covered Lines: 8544
Relevant Lines: 14339

💛 - Coveralls

Comment thread app/multieditmanager.cpp
@Withalion Withalion merged commit 552ece7 into master Dec 8, 2025
12 checks passed
@Withalion Withalion deleted the bugfix/fix-multiple-feature-deletion-bug branch December 8, 2025 10:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature to delete multiple selected features at once

3 participants