Skip to content
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

Issue592 reword filter content #697

Conversation

zheyichn
Copy link
Contributor

@zheyichn zheyichn commented Jun 14, 2024

  • Added one map for filter button actual option values -> display option values.
  • Filtering functions properly (see recording below).
  • TODO: My Prettier seems to do something differently then the existing formatter from main, many file changes are simply due to auto formatting, should I adjust my Prettier setting to make formatting more consistent ? (If so, could you pls share your Prettier setting with me @CodeWritingCow thx!)

Closes #592

owner-btn-local-test.mov

Copy link

vercel bot commented Jun 14, 2024

@zheyichn is attempting to deploy a commit to the Clean and Green Philly Team on Vercel.

A member of the Team first needs to authorize it.

@zheyichn zheyichn mentioned this pull request Jun 14, 2024
@@ -8,12 +8,14 @@ type ButtonGroupProps = {
options: string[];
selectedKeys: string[];
toggleDimension: (dimension: string) => void;
displayOptions?: { [key: string]: string };
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a good solution. It allows us to be flexible and change the button text without worrying about changing the button's value.

Copy link
Collaborator

@CodeWritingCow CodeWritingCow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@zheyichn the PR works as expected. That was a good, flexible solution that you came up with. As for Prettier, it looks like my code editor is formatting files the same way as yours. So I wouldn't worry too much about it for now. Thanks for your work!

@CodeWritingCow
Copy link
Collaborator

CodeWritingCow commented Jun 15, 2024

@zheyichn I've added this line to your PR description: Closes #592. For future PRs, let's tag the ticket number that you're working on in the PR description. It'll help everyone keep track of ticket statuses. Thanks!

@CodeWritingCow CodeWritingCow merged commit f87259e into CodeForPhilly:staging Jun 15, 2024
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants