Skip to content

Conversation

@feyyazcigim
Copy link
Contributor

This pull request refactors the SowOrderDialog component to introduce a clearer multi-step form flow, separating entry, review, and advanced parameter editing. It also adds new summary and tip estimation components for improved user feedback and modularity. The changes enhance usability, maintainability, and clarity of the sow order process.

Form Flow and State Management Improvements

  • Refactored the sow order form to use three steps: main entry, review, and advanced editing, replacing the previous operator tip step; added state management for draft advanced parameters and accordion UI. [1] [2] [3]
  • Implemented logic to set default values for advanced parameters (minSoil, maxPerSeason, podLineLength) based on user input and chain constants, improving user experience and reducing errors.

UI and Component Modularity

  • Introduced new shared components: SowOrderEntryFormParametersSummary, SowOrderFormAdvancedParametersSummary, and SowOrderFormButtonRow for modular review and advanced parameter display, and refactored the main dialog to use these. [1] [2]
  • Added SowOrderEstimatedTipPaid component to display estimated total tip paid, providing users with clearer cost expectations. [1] [2]

Codebase Cleanup and Usability

  • Updated imports and removed unused code, streamlined conditional rendering for the new multi-step flow, and improved button and error handling for better maintainability and user feedback. [1] [2] [3]

@feyyazcigim feyyazcigim requested a review from burr-nim December 7, 2025 23:33
@feyyazcigim feyyazcigim added the wip This issue or PR is under active development and may change frequently label Dec 7, 2025
@netlify
Copy link

netlify bot commented Dec 7, 2025

Deploy Preview for pintomoney ready!

Name Link
🔨 Latest commit c9e4274
🔍 Latest deploy log https://app.netlify.com/projects/pintomoney/deploys/693aee6feb0d0a0008f7d904
😎 Deploy Preview https://deploy-preview-337.preview.pinto.money
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@fr1jo
Copy link
Contributor

fr1jo commented Dec 8, 2025

@claude update the tooltip in the temperature tractor order form from The minimum Temperature at which this order can be executed. Temperature represents the current price relative to the moving average. to The minimum Temperature at which this order can be executed.

Update Execute during the Morning Auction to Execute during the Morning.
Update the Morning Tooltip from When enabled, this order will only execute during the Morning Auction period. to "The morning is the first 10 minutes of the Season, where the Temperature slowly increases to its maximum. Farmers can opt for their orders to execute during the Morning, such that their orders fill first.

@claude
Copy link
Contributor

claude bot commented Dec 8, 2025

Claude finished @fr1jo's task —— View job


✅ Updated Tractor Order Form Tooltips

Successfully updated the tooltip texts in the temperature tractor order form as requested.

Completed Tasks:

  • Search for current tooltip text to locate the right components
  • Update temperature tooltip to shorter version
  • Change "Execute during the Morning Auction" to "Execute during the Morning"
  • Update Morning tooltip with new detailed explanation
  • Commit and push changes

Changes Made:

  1. Temperature tooltip: Shortened from "The minimum Temperature at which this order can be executed. Temperature represents the current price relative to the moving average." to "The minimum Temperature at which this order can be executed."

  2. Morning label: Updated from "Execute during the Morning Auction" to "Execute during the Morning"

  3. Morning tooltip: Updated from "When enabled, this order will only execute during the Morning Auction period." to "The morning is the first 10 minutes of the Season, where the Temperature slowly increases to its maximum. Farmers can opt for their orders to execute during the Morning, such that their orders fill first."

All updates have been applied to src/components/Tractor/form/SowOrderV0Fields.tsx:39-45 and line 463 and committed to the PR branch.


claude bot and others added 14 commits December 8, 2025 01:48
- Shorten temperature tooltip to remove redundant explanation
- Change 'Execute during the Morning Auction' to 'Execute during the Morning'
- Update morning tooltip with detailed explanation about 10-minute period

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-authored-by: fr1jo <fr1jo@users.noreply.github.com>
Extract the select function outside the useCultivationFactor hook to maintain
a stable reference and prevent unnecessary re-renders. This improves performance
by avoiding function recreation on every render.

Co-authored-by: fr1jo <fr1jo@users.noreply.github.com>
@fr1jo
Copy link
Contributor

fr1jo commented Dec 11, 2025

@claude Update the Morning Tooltip from When enabled, this order will only execute during the Morning Auction period. to The morning is the first 10 minutes of the Season, where the Temperature slowly increases to its maximum. Farmers can opt for their orders to execute during the Morning, such that their orders fill first.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 11, 2025

Claude finished @fr1jo's task —— View job


✅ Updated Morning Tooltip Text

Todo List:

  • Search for current Morning tooltip text in the codebase
  • Update the tooltip from old text to new detailed text
  • Commit and push changes to the PR branch

Successfully updated the Morning tooltip text in the tractor order form as requested.

Change Made:
Updated the morningAuction tooltip in src/components/Tractor/form/SowOrderV0Fields.tsx:43 from:

"When enabled, this order will only execute during the Morning Auction period."

To:

"The morning is the first 10 minutes of the Season, where the Temperature slowly increases to its maximum. Farmers can opt for their orders to execute during the Morning, such that their orders fill first."

The change has been committed and pushed to the tractor-order-refactor branch.


github-actions bot and others added 2 commits December 11, 2025 00:55
Co-authored-by: fr1jo <fr1jo@users.noreply.github.com>
🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@fr1jo
Copy link
Contributor

fr1jo commented Dec 11, 2025

@claude

  1. update the morning tool tip and capitalize "morning" ("The Morning is the first 10 minutes of the Season, where the Temperature slowly increases to its maximum. Farmers can opt for their orders to execute during the Morning, such that their orders fill first."). Add an newline after "...increases to its maximum".

  2. update the Tip per Execution tooltip to go from "The tip per execution of the Sow order." to "the amount of PINTO paid to the operator when the order is executed."

@github-actions
Copy link
Contributor

github-actions bot commented Dec 11, 2025

Claude finished @fr1jo's task —— View job


✅ Updated Tooltip Texts

Todo List:

  • Search for current morning tooltip text in the codebase
  • Update morning tooltip with capitalization and newline formatting
  • Search for "Tip per Execution" tooltip text
  • Update "Tip per Execution" tooltip text
  • Commit and push changes to the PR branch

Changes Made:

  1. Morning Tooltip (src/components/Tractor/form/SowOrderV0Fields.tsx:43-44):

    • Capitalized "morning" to "Morning"
    • Added newline after "...increases to its maximum"
    • Updated from: "The morning is the first 10 minutes of the Season, where the Temperature slowly increases to its maximum. Farmers can opt for their orders to execute during the Morning, such that their orders fill first."
    • Updated to: "The Morning is the first 10 minutes of the Season, where the Temperature slowly increases to its maximum.\nFarmers can opt for their orders to execute during the Morning, such that their orders fill first."
  2. Tip per Execution Tooltip (src/components/Tractor/form/fields/sharedFields.tsx:226):

    • Updated from: "The tip per execution of the Sow order."
    • Updated to: "The amount of PINTO paid to the operator when the order is executed."

Both tooltip updates have been committed and pushed to the tractor-order-refactor branch.


github-actions bot and others added 2 commits December 11, 2025 01:13
- Capitalize 'Morning' and add newline to morning tooltip
- Update 'Tip per Execution' tooltip to clarify PINTO payment

Co-authored-by: fr1jo <fr1jo@users.noreply.github.com>
🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

wip This issue or PR is under active development and may change frequently

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants