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
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
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!