Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Source Pipedrive - Missing Product Tables + relations #19008

Closed
emeagenciadigital opened this issue Nov 5, 2022 · 15 comments
Closed

Source Pipedrive - Missing Product Tables + relations #19008

emeagenciadigital opened this issue Nov 5, 2022 · 15 comments
Assignees
Labels

Comments

@emeagenciadigital
Copy link

emeagenciadigital commented Nov 5, 2022

Tell us about the problem you're trying to solve

What are you trying to do, and why is it hard? A clear and concise description of what the problem is. Ex. I'm always frustrated when [THere is no way to get deal products, person related fields and deal related fields]

Describe the solution you’d like

  1. Add the followign missing content fron the following api endpoints:

    For missing products in deals:

The next to endpoints will help discovering the relation between a deal and a product.
- https://developers.pipedrive.com/docs/api/v1/Deals#getDealProducts
- https://developers.pipedrive.com/docs/api/v1/Products#getProductDeals

  1. For missing persons relations with custom fields:
    The "persons fields" table should contain the field pipedrive id, wich is named as KEY.
    The "persons" table should columnized the custom fields KEY.
@sajarin sajarin changed the title Pipedrive Missing Needed Tables Source Pipedrive - Missing Product Tables + relations Nov 7, 2022
@sajarin
Copy link
Contributor

sajarin commented Nov 7, 2022

Thanks for the issue @emeagenciadigital, are you willing to create a PR with the added streams + missing relations?

@emeagenciadigital
Copy link
Author

@sajarin I don't have the knowledge to do it myself. If it helps, for reference the singer tap of Pipedrive includes the following streams :

  • Deal flow
  • Deal products

@vidaletti
Copy link

@YowanR I'd like to work on this issue

@YowanR
Copy link
Contributor

YowanR commented Jan 13, 2023

Any progress on this, @vidaletti? Fixed the reward for this issue as well :)

@YowanR YowanR added the bounty label Jan 13, 2023
@vidaletti
Copy link

Hi @YowanR I'm already doing local tests. I believe next week I can release the PR

@emeagenciadigital
Copy link
Author

emeagenciadigital commented Jan 15, 2023 via email

@vidaletti
Copy link

vidaletti commented Jan 16, 2023

hi @YowanR / @sajarin Can you check if I have permission to create the branch? Button "create branch" is not enabled? Thanks!

@YowanR
Copy link
Contributor

YowanR commented Jan 31, 2023

@natalyjazzviolin @sajarin can you take a look, please?

@sajarin
Copy link
Contributor

sajarin commented Jan 31, 2023

hey @vidaletti you should be able to fork the airbyte repo and push changes to a branch in your forked copy and then make a PR on our repo based on that branch. Is this the process you're using?

@vidaletti
Copy link

@sajarin yes, follow the documentation for developers and I did as below. But the return is a 403
svidaletti@Sirlenos-MacBook-Pro airbyte % git push origin pipedrive-deals-19008
remote: Permission to airbytehq/airbyte.git denied to vidaletti.
fatal: unable to access 'https://github.com/airbytehq/airbyte.git/': The requested URL returned error: 403

@sajarin
Copy link
Contributor

sajarin commented Feb 1, 2023

@vidaletti you should push the pipesrive-deals-19008 branch to your forked repo and then navigate to the create pull request for airbytehq/airbyte and you should be able to create a PR. Make sure you enable commit access for maintainers when creating your PR. Hope this helps.

@vidaletti
Copy link

@sajarin and @YowanR I'm sorry! I had already done the fork and made the wrong local directory and branched over the wrong repo clone. Thanks for the help and sorry for the distraction

@emeagenciadigital
Copy link
Author

Hi @YowanR any progress on this ? Let me know if you need testers

@grubberr
Copy link
Contributor

In this PR #23789
I have implemented 3 additional streams discussed in this issue: Products, ProductFields, DealProducts

@grubberr
Copy link
Contributor

no response from customer I think we can close it

@marcosmarxm marcosmarxm added the contributor-program PRs submitted through the contributor program. label Apr 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Status: Unclaimed
Development

Successfully merging a pull request may close this issue.

7 participants