Skip to content

Conversation

@rformato
Copy link
Contributor

rename taskId to avoid conflicts with other libraries

Fixes #660

Description

Renamed the taskId variable to a less generic name

Steps to verify

Install react-native-auth0 and react-native-app-auth and check it compiles on iOS

rename taskId to avoid conflicts with other libraries
@rformato rformato changed the title Update RNAppAuth.m Fix a potential iOS build error due to a generic variable name that can be easily found in other projects (ie Auth0) Dec 29, 2021
@jpdriver jpdriver self-requested a review December 30, 2021 17:17
Copy link
Contributor

@jpdriver jpdriver left a comment

Choose a reason for hiding this comment

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

nice catch @rformato -- thanks for the PR! 👍🏻

@jpdriver jpdriver merged commit cceee6f into FormidableLabs:main Dec 30, 2021
gispada added a commit to gispada/react-native-app-auth that referenced this pull request Jan 4, 2022
kadikraman pushed a commit that referenced this pull request Jan 31, 2022
* Add OIDC RP Initiated Logout

* Rename taskId variable to comply with PR #697

* Add logout example for Okta

* Fix error in Okta logout example

Co-authored-by: Gianluca Spada <[email protected]>
Co-authored-by: Gianluca Spada <[email protected]>
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.

Error when trying to compile on iOS

2 participants