-
Notifications
You must be signed in to change notification settings - Fork 110
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
Feedback Form: Render it within both dashboard templates instead of redirecting #984
Comments
@kasugaijin this too |
@Aaryanpal how's this going? |
WIP |
@Aaryanpal no worries! Thanks for the update. |
Hi @kasugaijin, I'd like to propose a refactoring of our current feedback feature implementation. Here's an overview of the current situation and my proposed changes: Current Implementation:
Proposed Changes:
Rationale:
Please let me know your thoughts on this proposed refactoring. I'm happy to provide more detailed implementation plans if needed. |
Hey @Aaryanpal I appreciate the initiative! I think what you propose makes sense. The ony thing to handle differently if we use a single controller is the layout - we will want to use the dashboard layout if the link in the footer is clicked, and the adopter foster layout if the link in the adopter foster dash side nav if clicked...easy enough to handle with params and a before_action on the controller. |
For Staff it will look like thisFor Adopter it look something like thisHere's a rephrased version as a message to a dev: I've noticed that the two sections have different styles in the layout. To better align them with the overall design, do you think it would be better to create a separate controller to manage the styles more cleanly? Or should we handle the styling directly within the ERB? Also, do you think it makes sense to adjust the position of the "Feedback" heading to the top-left for better alignment with the design and remove the sub-heading ( Let me know your thoughts! |
- Added Feedback Controller and Feedback Mailer To Handle Feedback for staff and adopter - Remove DevContactsController and DevContactsMailer - Fix the test
- Added Feedback Controller and Feedback Mailer To Handle Feedback for staff and adopter - Remove DevContactsController and DevContactsMailer - Fix the test
Yeah that's what I was thinking in the original issue description - that two controllers would handle this more cleanly. Ideally, we do not have the Feedback Form title in the center on these dashboard scoped pages. The title should live in the header for each dashboard check the other pages under both dashboards to see how their headers are set with breadcrumbs). We should also move the shared components of the page (description and form) into a partial that can be shared between all three views (the form accessed via the footer, the form accessed via the two dashboards). |
We have the Feedback link in the Staff Dashboard, and the Adopter Foster Dashboard, but this redirects to a new page. Instead, we should render this form within each dashboard layout. You'll probably want two new controllers to handle this. Look at the
Organizations::AdopterFosterer::FaqController
as an example.The text was updated successfully, but these errors were encountered: