Skip to content

PM-4275 Integrate engagement description auto rewrite#1738

Merged
himaniraghav3 merged 3 commits intodevelopfrom
PM-4275
Mar 12, 2026
Merged

PM-4275 Integrate engagement description auto rewrite#1738
himaniraghav3 merged 3 commits intodevelopfrom
PM-4275

Conversation

@himaniraghav3
Copy link
Collaborator

Rewrites and updates the engagement description using jdAutowriteWorkflow

Editor is disabled while description is being generated

Button is disabled when there's no description

@himaniraghav3 himaniraghav3 requested a review from kkartunov March 12, 2026 08:30
}

const handleAIAutowrite = async () => {
if (isGeneratingDescription) return

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[⚠️ performance]
Consider adding a debounce mechanism to the handleAIAutowrite function to prevent multiple rapid clicks from triggering multiple requests. This can help reduce unnecessary API calls and improve performance.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We are disabling the button when it generates description so multiple clicks cannot be made

TC_FINANCE_API_URL: process.env.TC_FINANCE_API_URL || `${API_V6}/finance`,
TC_AI_API_BASE_URL: process.env.TC_AI_API_BASE_URL || `${API_V6}/ai`,
TC_AI_SKILLS_EXTRACTION_WORKFLOW_ID: process.env.TC_AI_SKILLS_EXTRACTION_WORKFLOW_ID || 'skillExtractionWorkflow',
TC_AI_AUTOWRITE_WORKFLOW_ID: process.env.TC_AI_AUTOWRITE_WORKFLOW_ID || 'jdAutowriteWorkflow',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add this also to production config.

@himaniraghav3 himaniraghav3 merged commit dab30b5 into develop Mar 12, 2026
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants