-
Notifications
You must be signed in to change notification settings - Fork 656
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
fix: Don't set ens filter to on sale by default on the current accoun… #2132
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
const EmotesFilters = [ | ||
...WearablesFilters.filter( | ||
filter => | ||
filter !== AssetFilter.BodyShape && | ||
filter !== AssetFilter.Network && | ||
filter !== AssetFilter.More && | ||
filter !== AssetFilter.OnlySmart | ||
), | ||
AssetFilter.Rarity, | ||
AssetFilter.Status, | ||
AssetFilter.Price, | ||
AssetFilter.Collection, | ||
AssetFilter.Creators, | ||
AssetFilter.OnSale, | ||
AssetFilter.More, | ||
AssetFilter.PlayMode |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added the more filter so it shows the on sale filter on my account for emotes.
Removed the .filter and copied the required filters for emotes to prevent the case in which new filters are added to the wearable filters and we forget to remove it from the emote filters.
// Only if the user is not in their own page, show ens on sale by default. | ||
if (window.location.pathname !== locations.currentAccount()) { | ||
newOptions.onlyOnSale = | ||
previous.onlyOnSale === undefined ? true : current.onlyOnSale | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What was happening is that on your own account page, it was being set to only on sale, which then forced the only on sale on the other tabs.
…t page
Closes #2129