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
feat: filter proposals #5664
feat: filter proposals #5664
Conversation
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.
Just left some minor comments. Works well, good job!
packages/shared/locales/en.json
Outdated
"upcoming": "Announcement", | ||
"commencing": "Voting open", | ||
"holding": "Counting starts", | ||
"ended": "Counting stops" |
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.
Is it possible to use the same locale's as we use for the labels?
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.
wrt translations for filters, the structure isnt that flexible...
phase: { | ||
active: false, | ||
type: 'selection', | ||
localeKey: 'filters.phase', |
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.
Can we update this then to alleviate the burden on the translaters and to keep sure it's consistent across all languages? I guess the only thing that would need add is a label property to proposalStatus
?
localeKey: 'filters.phase', | |
localeKey: 'proposalStatus', |
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.
LGTM! 🔥
Summary
This PR includes a filter popup to filter all proposals for various fields
Changelog
Relevant Issues
Clsoes #4870
Testing
Platforms
Instructions
...
Checklist