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

Replace uses of "dropdown" with respective ViewComponent #5845

Open
elasticspoon opened this issue Jun 19, 2024 · 6 comments · May be fixed by #5907
Open

Replace uses of "dropdown" with respective ViewComponent #5845

elasticspoon opened this issue Jun 19, 2024 · 6 comments · May be fixed by #5907
Labels
Difficulty: Intermediate Spans multiple domains or requires more advanced coding knowledge no-issue-activity

Comments

@elasticspoon
Copy link
Collaborator

What type(s) of user does this feature affect?

Steve Ballmer DEVELOPERS DEVELOPERS DEVELOPERS

Description

This application uses ViewComponent. While we have made multiple components we haven't replaced old code in all the place where those view components could be used.

Find the places that implement dropdowns but don't use the dropdown component (https://github.com/rubyforgood/casa/blob/main/app/components/dropdown_menu_component.rb). Replace them.

This should be a good place to start with replacements: https://github.com/search?q=repo%3Arubyforgood%2Fcasa+AND+path%3A*.html.erb+AND+%2Fdropdown%5B%22%7C+%5D%2F&type=code

How to access the QA site

Login Details:
Link to QA site

Login Emails:

password for all users: 12345678

Questions? Join Slack!

We highly recommend that you join us in slack #casa channel to ask questions quickly. And discord for office hours (currently Tuesday 5-7pm Pacific), stakeholder news, and upcoming new issues.

@iamronakgupta
Copy link
Contributor

can i take this up?

Copy link

This issue has been inactive for 262 hours (10.92 days) and will be unassigned after 98 more hours (4.08 days). If you have questions, please

If you are still working on this, comment here to tell the bot to give you more time

@iamronakgupta
Copy link
Contributor

still working on this issue

@elasticspoon elasticspoon added the Difficulty: Intermediate Spans multiple domains or requires more advanced coding knowledge label Jul 7, 2024
Copy link

This issue has been inactive for 260 hours (10.83 days) and will be unassigned after 100 more hours (4.17 days). If you have questions, please

If you are still working on this, comment here to tell the bot to give you more time

Copy link

This issue has been inactive for 380 hours (15.83 days) and is past the limit of 360 hours (15.00 days) so is being unassigned.You’ve just been unassigned from this ticket due to inactivity – but feel free to pick it back up (or a new one!) in the future! Thank you again for your contribution to this project.

Copy link

This issue has been open without changes for a long time! What's up?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Difficulty: Intermediate Spans multiple domains or requires more advanced coding knowledge no-issue-activity
Projects
Status: In progress
Development

Successfully merging a pull request may close this issue.

2 participants