Adding param to screens fragments 🍖 #518
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description of change ✍️
Adding iconFragment to screens since there might be multiple icons files in other SNE apps
Priority 🥇
Medium - we can override these locally, but they'd be nice to have in the FFL
Effect on other applications using FFB 🌊
No breaking changes - this is adding a new param to a fragment, if it's null, it will not show up
✅ Checklist before requesting a review
*Didn't update docs/tests/readme since
iconName
doesn't explicitly have these either