Skip to content

fix(vtiger): fetch module list instead of using static list and improve custom api call#8523

Merged
onyedikachi-david merged 6 commits intoactivepieces:mainfrom
privatestefans:fix/pieces/vtiger
Aug 13, 2025
Merged

fix(vtiger): fetch module list instead of using static list and improve custom api call#8523
onyedikachi-david merged 6 commits intoactivepieces:mainfrom
privatestefans:fix/pieces/vtiger

Conversation

@privatestefans
Copy link
Copy Markdown
Contributor

@privatestefans privatestefans commented Jul 25, 2025

What does this PR do?

  • The static module list has now been replaced with a dynamic list, fetched from the API
  • Mandatory fields are now prioritized at the top to help users quickly identify which fields must be filled
  • Add URL to Custom API Call to allow user using API path
  • Some legacy module list remains undeleted/changed to ensure compatibility with previous behavior

Explain How the Feature Works

Vtiger.mp4

Relevant User Scenarios

/claim #8485
Fixes #8485

@onyedikachi-david
Copy link
Copy Markdown
Collaborator

@privatestefans please do these as specified in the issue:

Steps to solve:
Start working: Comment /attempt #8485 with your implementation plan
Submit work: Create a pull request including /claim #8485 in the PR body to claim the bounty

@privatestefans
Copy link
Copy Markdown
Contributor Author

@onyedikachi-david Ahh yes. Thanks for reminding me. I will do it 👍

@onyedikachi-david onyedikachi-david self-requested a review August 13, 2025 17:49
* upstream/main: (6611 commits)
  chore: auto bump translated pieces
  New translations translation.json (Chinese Traditional)
  New translations translation.json (Chinese Traditional)
  New translations translation.json (Chinese Traditional)
  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 Simplified)
  New translations translation.json (Chinese Simplified)
  New translations translation.json (Chinese Simplified)
  New translations translation.json (Chinese Traditional)
  New translations translation.json (Chinese Traditional)
  New translations translation.json (Chinese Traditional)
  New translations translation.json (Chinese Traditional)
  New translations translation.json (Chinese Traditional)
  New translations translation.json (Chinese Traditional)
  New translations translation.json (Chinese Simplified)
  ...
@nx-cloud
Copy link
Copy Markdown

nx-cloud bot commented Aug 13, 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 f504f01

Command Status Duration Result
nx run-many --target=build -c production --proj... ❌ Failed 29s View ↗
nx affected --target=build -c production --excl... ✅ Succeeded <1s View ↗
nx run-many --target=lint --projects=pieces-vti... ✅ Succeeded 2m 31s View ↗
nx affected --target=lint --exclude=pieces-* --... ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2025-08-13 18:20:13 UTC

@onyedikachi-david onyedikachi-david merged commit 9dcdc25 into activepieces:main Aug 13, 2025
9 of 10 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.

Improvement/expanding of the VtigerCRM API (MC)Piece ;)

3 participants