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.

Sign up for GitHub

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

Jump to bottom

Fix User.update() method #145

Closed
wants to merge 1 commit into from
Closed

Fix User.update() method #145

wants to merge 1 commit into from

Conversation

Saluev
Copy link

@Saluev Saluev commented Nov 8, 2022 •
edited
Loading

I assume it should be this way because inherited PNResource.is_update() method checks _update field.

Setting _update = True manually allowed me to solve 403 issue with my app.

@Saluev
Fix User.update() method
dad043f 
I assume it should be this way because inherited `PNResource.is_update()` method checks `_update` field.
seba-aln added a commit that referenced this pull request Nov 21, 2022
@seba-aln
Fix typo in consumer models
d864742 
Closes #145
@seba-aln seba-aln mentioned this pull request Nov 21, 2022
Merged
seba-aln added a commit that referenced this pull request Nov 24, 2022
@seba-aln @client-engineering-bot
Fixes for various errors (#146)
8ee850b 
* Fix typo in consumer models

Closes #145

* Add urls to PyPi package

Closes #115

Fix for is_error

Closes #102

* Typo and a shorthand

* PubNub SDK 7.0.2 release.

Co-authored-by: Client Engineering Bot <60980775+client-engineering-bot@users.noreply.github.com>
@pn-op
Copy link

pn-op commented Nov 24, 2022

@Saluev this issue is addressed in 7.0.2

@github-actions github-actions bot added the status: done This issue is considered resolved. label Nov 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@seba-aln seba-aln Awaiting requested review from seba-aln seba-aln is a code owner

@kleewho kleewho Awaiting requested review from kleewho

@marek-lewandowski marek-lewandowski Awaiting requested review from marek-lewandowski

Assignees
No one assigned
Labels
status: done This issue is considered resolved.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Saluev @pn-op