Skip to content

Conversation

@diwu-sf
Copy link
Contributor

@diwu-sf diwu-sf commented Apr 25, 2025

Pipedrive source should have projects and tasks
https://developers.pipedrive.com/docs/api/v1/Projects
https://developers.pipedrive.com/docs/api/v1/Tasks

The recents documentation doesn't make mention of these objects because it's an addon:
https://support.pipedrive.com/en/article/projects-by-pipedrive

VSCode also did removals of all unused imports to prevent lint complaints

@diwu-sf
Copy link
Contributor Author

diwu-sf commented May 14, 2025

@rudolfix can you take a look please?

@rudolfix rudolfix added the ci from fork Allows to run tests from PR coming from fork label May 19, 2025
Copy link
Contributor

@rudolfix rudolfix left a comment

Choose a reason for hiding this comment

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

LGTM! thanks!

@rudolfix rudolfix merged commit 85e8f7d into dlt-hub:master May 19, 2025
7 of 13 checks passed
@diwu-sf diwu-sf deleted the diwu-update-pipedrive branch May 19, 2025 20:43
evidy-bot added a commit to evidycz/verified-sources that referenced this pull request Jun 2, 2025
* fix: tests import exception from correct location (dlt-hub#613)

* exception moved

* bump dltpure dependency version

* Fix wrong initialization command on Stripe documentation (dlt-hub#618)

* docs(inbox): GMail concurrency (dlt-hub#615)

* fix(inbox): senders filter is optional (dlt-hub#614)

* fixes forks test action

* fix(inbox): RFC 2822 dates parsing (dlt-hub#616)

* update pipedrive source with projects and tasks (dlt-hub#612)

* docs(inbox): remove mention of unused dependencies (dlt-hub#617)

* change contribution guide (dlt-hub#620)

* update contributing docs

* bumps uv lock

* fixes some tests

* Update CONTRIBUTING.md

Co-authored-by: anuunchin <88698977+anuunchin@users.noreply.github.com>

---------

Co-authored-by: anuunchin <88698977+anuunchin@users.noreply.github.com>

---------

Co-authored-by: djudjuu <julius@dlthub.com>
Co-authored-by: Pedro <pdfrod@users.noreply.github.com>
Co-authored-by: Joscha Feth <joscha@feth.com>
Co-authored-by: Marcin Rudolf <rudolfix@rudolfix.org>
Co-authored-by: diwu-sf <di.wu@shadowfaxdata.com>
Co-authored-by: anuunchin <88698977+anuunchin@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci from fork Allows to run tests from PR coming from fork

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants