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

Merge latest changes in main to analytics-models feature-dev branch #119

Open
wants to merge 28 commits into
base: feature-dev/analytics-models
Choose a base branch
from

Conversation

rushabhvaria
Copy link
Member

No description provided.

thucngyyen and others added 27 commits February 15, 2023 16:20
* Update generated client to 0.1.7

* Fix null configs for converion_tag

* Linting

* Feature fix/conversion tag configs (#83)

* Linting

* Update test
Calling <entity>.get_all() would return a list of entities and a Bookmark model that returns a new entity_list on each Bookmark.get_next() call which performs API calls only when necessary to fetch a new set, and eventually returns all entities for the parent.
As a result of the `PinterestBaseModel.__init__` getting called without the passed in client from the `Ad` model constructor, the requests failed resulting in an Auth error. This PR should fix the problem.
As integration_tests was a folder too, a PHONY target needed to be added for the makefile target.
* Update conversion tag

* Update conversion event

* Update pin model
* fix: test board

* fix: test board

* fix: change media to base64

* fix: typo random  (#101)

* fix: typo

* Fix: typo
* Fix integration test set up

* Fix lint error
* Fix bug for following bookmark on list_pins in Board model

* Don't pass params if cls present
* Generate docs 0.2.1

* Update doc versioning
…123)

* fix to incorrect error message when wrong refresh token is passed in

* integration test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants