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

Create list using list comprehension #644

Open
wants to merge 7 commits into
base: release/v1.12.6
Choose a base branch
from

Conversation

ericbn
Copy link
Contributor

@ericbn ericbn commented Apr 21, 2023

instead of creating an empty list and appending to it.

ericl-billgo and others added 7 commits April 20, 2023 11:04
* Bug/pyjwt dependency (simple-salesforce#610)

* Update setup.py

Update pyjwt dependency

* Update setup.py

include cryptography for requirement for pyjwt

* Update setup.py

include pyjwt[crypto] in install requirements

* Update setup.py

remove [crypto] from pyjwt[crypto] in install_requires as causes travis-ci failure to build

* remove extra leading space

---------

Co-authored-by: JonWobken <49450080+jon-wobken@users.noreply.github.com>
…s_hook (simple-salesforce#634)

* Bug/pyjwt dependency (simple-salesforce#610)

* Update setup.py

Update pyjwt dependency

* Update setup.py

include cryptography for requirement for pyjwt

* Update setup.py

include pyjwt[crypto] in install requirements

* Update setup.py

remove [crypto] from pyjwt[crypto] in install_requires as causes travis-ci failure to build

* Updated __getattr__ function to create SFType object with object_pairs_hook as input

Right now let us say that the user creates a SFType object using __getattr__ function which is present in the class SalesForce. If the SalesForce object is initiated using object_pairs_hook being dict or print. In that case the user would want the response from SFType to be same as that object as well. So I have added object_pairs_hook in the SFType object which is being called in the __getattr__ function of the SalesForce Class

* Updated __getattr__ function to create SFType object with object_pairs_hook

Right now let us say that the user creates a SFType object using getattr function which is present in the class SalesForce. If the SalesForce object is initiated using object_pairs_hook being dict or print. In that case the user would want the response from SFType to be same as that object as well. So I have added object_pairs_hook in the SFType object which is being called in the getattr function of the SalesForce Class

* removed whitespaces

* pylint

---------

Co-authored-by: JonWobken <49450080+jon-wobken@users.noreply.github.com>
* Bug/pyjwt dependency (simple-salesforce#610)

* Update setup.py

Update pyjwt dependency

* Update setup.py

include cryptography for requirement for pyjwt

* Update setup.py

include pyjwt[crypto] in install requirements

* Update setup.py

remove [crypto] from pyjwt[crypto] in install_requires as causes travis-ci failure to build

* Fix indent in setup.py

---------

Co-authored-by: JonWobken <49450080+jon-wobken@users.noreply.github.com>
* Bug/pyjwt dependency (simple-salesforce#610)

* Update setup.py

Update pyjwt dependency

* Update setup.py

include cryptography for requirement for pyjwt

* Update setup.py

include pyjwt[crypto] in install requirements

* Update setup.py

remove [crypto] from pyjwt[crypto] in install_requires as causes travis-ci failure to build

* fix: use unescaped username for JWT claim

JWT Bearer Flow needs JWT claim sub containg unescaped username
Connected App Login has been fixed by simple-salesforce#604

---------

Co-authored-by: JonWobken <49450080+jon-wobken@users.noreply.github.com>
instead of creating an empty list and appending to it.
@ericbn ericbn changed the title Create list using generator Create list using list comprehension Apr 23, 2023
@jon-wobken jon-wobken changed the base branch from release/v1.12.4 to release/v1.12.6 October 24, 2023 12:04
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