-
Notifications
You must be signed in to change notification settings - Fork 744
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
WIP: Add the ability to create a new Strategy or Moment from an individual Category page #2330
base: main
Are you sure you want to change the base?
Conversation
Added the buttons for creating new Moments and Strategies to the Categories page. They currently do not go anywhere meaning that the next commit needs to correctly handle routing so that the New Moment and New Strategy pages appear successfully.
Thank you for opening this pull request with us! Be sure to follow our Pull Request Practices. Let us know if you have any questions on Slack. |
Hey @lenikadali do you need any help here? |
Hi 👋🏿 Thanks for checking in here 🙌🏿 The plan is to try out your suggestions either today or tomorrow and see how it goes 🤞🏿 |
Added form props for Moment and Strategy to the categories_form_helper
Oh. Then what did this statement mean in the issue description 🤔?
Also, I noticed that the documentation for the version used of React on Rails is now not easily accessible. Maybe it could be worth it to first upgrade React on Rails as per this ticket and then attempt this one since it would be easier to reference the documentation as we work on this. |
Ohhh sorry I think I misunderstood / forgot about this ticket was asking for even though I wrote it haha. Yes so what you want to do is modify the form for the |
Yeah, we should upgrade React on Rails! It doesn't block this work, but I agree! There's a ticket to do that already with the latest version of React. #2115 |
Description
Adds the ability to create a new Strategy or Moment from an individual Category page.
Corresponding Issue
#2300