Skip to content

fix: Crop Planner Not Changing Based on Input - #1793

Merged
omroy07 merged 3 commits into
omroy07:mainfrom
Aditya8369:301
May 28, 2026
Merged

fix: Crop Planner Not Changing Based on Input#1793
omroy07 merged 3 commits into
omroy07:mainfrom
Aditya8369:301

Conversation

@Aditya8369

Copy link
Copy Markdown
Contributor

The crop planner no longer hardcodes Tomato. I replaced the placeholder path in script.js with a local scoring engine that ranks multiple crops using pH, temperature, rainfall, soil type, season, market demand, fertilizer, pest issue, and irrigation method, then renders a crop-specific guide from the winning match. I also neutralized the legacy duplicate submit handler so it can’t overwrite the result after the new recommendation is shown; the guard is at script.js.

Validation passed with node --check and a static error scan on the touched file.

closes #301

@vercel

vercel Bot commented May 25, 2026

Copy link
Copy Markdown

@Aditya8369 is attempting to deploy a commit to the Om Roy's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions

Copy link
Copy Markdown

Thanks for creating a PR for your Issue! ☺️

We'll review it as soon as possible.
In the meantime, please double-check the file changes and ensure that all commits are accurate.

If there are any unresolved review comments, feel free to resolve them. 🙌🏼

@Aditya8369

Copy link
Copy Markdown
Contributor Author

@omroy07 please merge under NSOC

@omroy07
omroy07 merged commit dd64e22 into omroy07:main May 28, 2026
1 of 4 checks passed
@github-actions

Copy link
Copy Markdown

🎉 Congrats @Aditya8369 on getting your PR merged! 🙌
Thanks for the contribution — looking forward to more from you 🚀

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.

Crop Planner Not Changing Based on Input

2 participants