Skip to content

feat(zendesk-sell): add piece#9763

Merged
onyedikachi-david merged 7 commits intoactivepieces:mainfrom
Pranith124:feat/zendesk-sell
Oct 27, 2025
Merged

feat(zendesk-sell): add piece#9763
onyedikachi-david merged 7 commits intoactivepieces:mainfrom
Pranith124:feat/zendesk-sell

Conversation

@Pranith124
Copy link
Copy Markdown
Contributor

@Pranith124 Pranith124 commented Oct 18, 2025 •
edited
Loading

What does this PR do?

Explain How the Feature Works

Relevant User Scenarios

Demo -actions

zendesk-sell.actions.mp4

Demo -triggers

zendesk-triggerss.mp4

Fixes #9754
/claim #9754

@algora-pbc algora-pbc bot mentioned this pull request Oct 18, 2025
Closed
@Pranith124
Copy link
Copy Markdown
Contributor Author

Pranith124 commented Oct 18, 2025 •
edited
Loading

Hey @kishanprmr @sanket-a11y @onyedikachi-david,

This PR is now ready for review. I wasn’t able to upload the demo video since the API key requires a paid account.

I’ve sent you a DM on Discord, @kishanprmr — if you could please share the credentials, I’ll upload the live demo showcasing the implementation of actions and triggers.

Thank you so much! 🙏

@sanket-a11y sanket-a11y self-assigned this Oct 20, 2025
@nx-cloud
Copy link
Copy Markdown

nx-cloud bot commented Oct 26, 2025 •
edited
Loading

🤖 Nx Cloud AI Fix Eligible

An automatically generated fix could have helped fix failing tasks for this run, but Self-healing CI is disabled for this workspace. Visit workspace settings to enable it and get automatic fixes in future runs.

To disable these notifications, a workspace admin can disable them in workspace settings.

View your CI Pipeline Execution ↗ for commit 4ce0dd4

Command Status Duration Result
nx run server-api:test-ce ❌ Failed 1m 43s View ↗
nx run server-api:test-cloud ✅ Succeeded 21s View ↗
nx run-many --target=test --projects=engine,shared ✅ Succeeded 11s View ↗
nx run server-api:test-ee ✅ Succeeded 10s View ↗
nx run-many --target=build -c production --proj... ✅ Succeeded 4s View ↗
nx affected --target=build -c production --excl... ✅ Succeeded <1s View ↗
nx run-many --target=lint --projects=pieces-zen... ✅ Succeeded 1s View ↗
nx affected --target=lint --exclude=pieces-* ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2025-10-27 09:20:13 UTC

@onyedikachi-david onyedikachi-david self-requested a review October 26, 2025 18:50
@onyedikachi-david onyedikachi-david self-assigned this Oct 26, 2025
@onyedikachi-david
Copy link
Copy Markdown
Collaborator

onyedikachi-david commented Oct 26, 2025

Hi @Pranith124, thank you for a detailed contribution, was about merging, but on a second thought, I will request you refactor triggers to make use of Polling as TriggerStrategy. Ensure to use Polling helpers.

@Pranith124
Copy link
Copy Markdown
Contributor Author

Pranith124 commented Oct 26, 2025 •
edited
Loading

Yeah @onyedikachi-david , I’ll update the trigger to use the polling strategy.

@Pranith124
Copy link
Copy Markdown
Contributor Author

Pranith124 commented Oct 26, 2025

Hey @onyedikachi-david, I just updated the triggers. Let me know if you’d like any changes — I can update them right away.

@onyedikachi-david onyedikachi-david self-requested a review October 27, 2025 04:33
@onyedikachi-david onyedikachi-david merged commit ac585df into activepieces:main Oct 27, 2025
15 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@onyedikachi-david onyedikachi-david onyedikachi-david approved these changes

Assignees

@onyedikachi-david onyedikachi-david

@sanket-a11y sanket-a11y

Labels

🧩 area/third-party-pieces 🙋 Bounty claim

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[MCP] Zendesk Sell

3 participants

@Pranith124 @onyedikachi-david @sanket-a11y