Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bug: "Delete" option is displayed for built-in rules despite being undeletable #69

Open
suvanl opened this issue Mar 9, 2024 · 0 comments
Assignees
Labels
bug Something isn't working priority: medium This is a medium-priority issue UI layer Relates to the app's UI layer

Comments

@suvanl
Copy link
Owner

suvanl commented Mar 9, 2024

App and system information

- FixMyLinks app version: 1.2.1
- Android version: 14 (API 34)
- Android build number: UE1A.230829.019
- Device model: Generic AVD (Medium Phone API 34)
- Device type: Emulated

Summary of expected behaviour

There should be no way to delete a built-in rule.

Summary of actual behaviour

It's possible to "delete" a built-in rule in the same way that a custom rule can be deleted, via RuleDetailsScreen.

The rule doesn't actually get deleted, but there's nothing to indicate that a built-in rule cannot be deleted, so users may expect the rule to have been deleted successfully.

Description of what you think should be changed

The overflow menu containing the "Delete" option on RuleDetailsScreen should only be displayed for custom rules, not built-in rules.

Steps to reproduce the bug

  1. Open the app
  2. Go to the "Rules" destination using the bottom navigation bar
  3. Tap the "Built-in" tab title, or swipe to it
  4. Tap on any built-in rule card
  5. Activate the overflow menu in the top right of the screen
  6. Tap "Delete"
  7. Tap "Delete" again on the confirmation dialog
  8. Observe that the app navigates back to RulesScreen, just like when a custom rule is deleted.
    • Unlike custom rules, the built-in rule doesn't actually get deleted. The rule not being deleted is a good thing, because it shouldn't be possible to delete built-in rules (and at this moment in time isn't technically possible, given that BIRs are hardcoded).

Additional info

No response

@suvanl suvanl added the unconfirmed bug Something might not be working, but we're not yet sure if it can be reproduced consistently label Mar 9, 2024
@suvanl suvanl self-assigned this Mar 9, 2024
@suvanl suvanl added bug Something isn't working UI layer Relates to the app's UI layer priority: medium This is a medium-priority issue and removed unconfirmed bug Something might not be working, but we're not yet sure if it can be reproduced consistently labels Mar 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working priority: medium This is a medium-priority issue UI layer Relates to the app's UI layer
Projects
None yet
Development

No branches or pull requests

1 participant