Baseflow / Baseflow/flutter-permission-handler
Remove unused constant
- Dominant language
- Dart
- Stars
- 2.2k
- Forks
- 970
- Avg merge
- 14h 22m
- Merged PRs (30d)
- 2
Description
## 🏗 Enhancement Proposal
Remove unused `PERMISSION_PHOTOS_ADD_ONLY` constant:
https://github.com/Baseflow/flutter-permission-handler/blob/2e47f05ae9dabee2bc671166301aa948259b39f2/permission_handler_apple/ios/Classes/PermissionHandlerEnums.h#L60
It is confusing as it suggest it should be used for `addOnly` photos permission, while it does nothing and `PERMISSION_PHOTOS` is needed.
### Platforms affected (mark all that apply)
- [x] :iphone: iOS
- [ ] :robot: Android
Contributor guide
Research direction
Open permission_handler_apple/ios/Classes/PermissionHandlerEnums.h at the linked constant and search the repository for references to PERMISSION_PHOTOS_ADD_ONLY. Remove the unused constant, then verify that the iOS plugin still builds and that PERMISSION_PHOTOS remains the value used for photo permissions.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- ios
- Domain
- mobile-dev
- Issue type
- Refactor
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 48/100