Now it checks if the preview screenshot URL is already in the list.
Sometimes it happens that preview1 and preview2 are the same URL.
Details
Details
- Reviewers
apol davidedmundson abetts - Group Reviewers
Discover Software Store - Commits
- R134:68c1aaabadf7: Removed duplicate preview screenshots
Before:
After:
Diff Detail
Diff Detail
- Repository
- R134 Discover Software Store
- Branch
- master
- Lint
No Linters Available - Unit
No Unit Test Coverage
Comment Actions
What do you mean?
In what situation would it add duplicates?
This change is to make sure it does not add duplicates.
Comment Actions
I think Aleix is saying that you should make the change inside appendIfValid() instead of duplicating 4 times the same code that calls appendIfValid().
Thanks for the patch, BTW!