Skip to content

feat: implemented bigin by zoho as piece#8677

Merged
onyedikachi-david merged 11 commits intoactivepieces:mainfrom
gs03-dev:pieces/zoho-bigin
Aug 11, 2025
Merged

feat: implemented bigin by zoho as piece#8677
onyedikachi-david merged 11 commits intoactivepieces:mainfrom
gs03-dev:pieces/zoho-bigin

Conversation

@gs03-dev
Copy link
Copy Markdown
Contributor

@gs03-dev gs03-dev commented Aug 6, 2025

What does this PR do?

This PR implements Bigin by Zoho as a piece on active pieces, please review.

/claim #8601
Fixes #8601

@gs03-dev
Copy link
Copy Markdown
Contributor Author

gs03-dev commented Aug 6, 2025

I would like to point out that the last search action cannot be implemented as the API of Bigin by Zoho does not provide a way to filter through Users by criteria/filter, in this case, email address.

I thought of a work around of fetching the data from users api, then use JS to find the actual user but i see that as overengineering and unneccessary because this becomes an issue and can make action slow if users data grows to an unprecented level

@gs03-dev
Copy link
Copy Markdown
Contributor Author

gs03-dev commented Aug 6, 2025

@kishanprmr and @onyedikachi-david please review

@gs03-dev gs03-dev changed the title Pieces/zoho bigin feat: implemented bigin by zoho as piece Aug 6, 2025
@algora-pbc algora-pbc bot mentioned this pull request Aug 6, 2025
@onyedikachi-david
Copy link
Copy Markdown
Collaborator

To claim a bounty, you need to provide a short demo video of your changes in your pull request

@onyedikachi-david
Copy link
Copy Markdown
Collaborator

I would like to point out that the last search action cannot be implemented as the API of Bigin by Zoho does not provide a way to filter through Users by criteria/filter, in this case, email address.

I thought of a work around of fetching the data from users api, then use JS to find the actual user but i see that as overengineering and unneccessary because this becomes an issue and can make action slow if users data grows to an unprecented level

Yes you can: https://www.bigin.com/developer/docs/apis/v2/get-users.html

@gs03-dev
Copy link
Copy Markdown
Contributor Author

gs03-dev commented Aug 7, 2025

I would like to point out that the last search action cannot be implemented as the API of Bigin by Zoho does not provide a way to filter through Users by criteria/filter, in this case, email address.

I thought of a work around of fetching the data from users api, then use JS to find the actual user but i see that as overengineering and unneccessary because this becomes an issue and can make action slow if users data grows to an unprecented level

Yes you can: https://www.bigin.com/developer/docs/apis/v2/get-users.html

Hello bro, the search action requires fetching users via email address. This endpoint doesn't do that

I saw this endpoint which is why I was talking about pulling all users and using JS to filter

@gs03-dev
Copy link
Copy Markdown
Contributor Author

gs03-dev commented Aug 7, 2025

To claim a bounty, you need to provide a short demo video of your changes in your pull request

I will upload a video

@gs03-dev
Copy link
Copy Markdown
Contributor Author

gs03-dev commented Aug 7, 2025

@gs03-dev
Copy link
Copy Markdown
Contributor Author

gs03-dev commented Aug 8, 2025

Please review @kishanprmr

1 similar comment
@gs03-dev
Copy link
Copy Markdown
Contributor Author

gs03-dev commented Aug 9, 2025

Please review @kishanprmr

* upstream/main: (3062 commits)
  fix: padding
  refactor: remove duplicated create agent
  refactor: clean up imports and improve code formatting
  refactor: remove testPrompt from agent model and related components
  refactor: project state
  fix: remove unused import statement in project-diff.service.ts
  feat: generate translation files for all pieces [skip-changelog] (activepieces#8738)
  chore: remove console log for loaded translations in i18n.ts
  fix: only push if there are changes from the bumping subaction
  Revert "New translation files (activepieces#8735)"
  New translation files (activepieces#8735)
  fix: make generate translation files for pieces action use correct github token
  fix: generating new pieces translations gh action
  fix: use github-token instead of env variable
  fix: lint
  chore: push commit after adding it for bumping translated pieces
  fix: auto merge pr even if no changes were detected from the bump action
  Revert "chore: auto bump translated pieces"
  chore: auto bump translated pieces
  fix(tables): create table ensuring table-agent created
  ...
@onyedikachi-david onyedikachi-david self-assigned this Aug 11, 2025
@nx-cloud
Copy link
Copy Markdown

nx-cloud bot commented Aug 11, 2025

🤖 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 3b1e9d6

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

☁️ Nx Cloud last updated this comment at 2025-08-11 06:54:33 UTC

@onyedikachi-david onyedikachi-david self-requested a review August 11, 2025 03:44
@onyedikachi-david onyedikachi-david self-requested a review August 11, 2025 06:54
* upstream/main: (783 commits)
  chore: auto bump translated pieces
  Revert "chore: auto bump translated pieces"
  chore: pull before merging for corwdin auto merge pr action
  chore: auto bump translated pieces
  fix: remove extra push to pr in crowdin auto merge action
  Revert "chore: auto bump translated pieces"
  chore: auto bump translated pieces
  New translations translation.json (Portuguese, Brazilian)
  New translations translation.json (Portuguese, Brazilian)
  New translations translation.json (Portuguese, Brazilian)
  New translations translation.json (Vietnamese)
  New translations translation.json (Vietnamese)
  New translations translation.json (Vietnamese)
  New translations translation.json (Chinese Traditional)
  New translations translation.json (Chinese Traditional)
  New translations translation.json (Chinese Traditional)
  New translations translation.json (Chinese Simplified)
  New translations translation.json (Chinese Simplified)
  New translations translation.json (Chinese Simplified)
  New translations translation.json (Chinese Traditional)
  ...
@onyedikachi-david onyedikachi-david merged commit ac10aaf into activepieces:main Aug 11, 2025
5 checks passed
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.

[MCP] Bigin by Zoho

2 participants