Skip to content

ref(naming): Rename user to publicKey in Dsn type and class - #3225

Merged
lobsterkatie merged 1 commit into
masterfrom
kmclb-user-in-dsn-is-public-key
Feb 2, 2021
Merged

ref(naming): Rename user to publicKey in Dsn type and class#3225
lobsterkatie merged 1 commit into
masterfrom
kmclb-user-in-dsn-is-public-key

Conversation

@lobsterkatie

@lobsterkatielobsterkatie commented Feb 1, 2021

Copy link
Copy Markdown
Member

... since that's the way we refer to it everywhere else.

I included backwards compatibility, but I'm torn on whether that's even necessary, since this is deep in the SDK's internals. Interested in opinions there. UPDATE: Kept the backwards compatibility under the hood, but removed the logging, so it adapts silently.

@github-actions

github-actionsBot commented Feb 1, 2021

Copy link
Copy Markdown
Contributor

size-limit report

PathSize
@sentry/browser - CDN Bundle (gzipped)20.38 KB (+0.17% 🔺)
@sentry/browser - Webpack21.27 KB (+0.17% 🔺)
@sentry/react - Webpack21.29 KB (+0.17% 🔺)
@sentry/browser + @sentry/tracing - CDN Bundle (gzipped)27.32 KB (+0.14% 🔺)

Comment threadpackages/utils/src/dsn.ts
@lobsterkatie
lobsterkatieforce-pushed the kmclb-user-in-dsn-is-public-key branch from 3c62321 to 9e6e3a4CompareFebruary 2, 2021 18:01
@lobsterkatielobsterkatie changed the title ref(naming): Rename user to publicKey in Dsn type and classref(naming): Rename user to publicKey in Dsn type and classFeb 2, 2021
@lobsterkatie
lobsterkatie merged commit 02d0ded into masterFeb 2, 2021
@lobsterkatie
lobsterkatie deleted the kmclb-user-in-dsn-is-public-key branch February 2, 2021 18:28
This was referenced Mar 7, 2021
This was referenced Mar 14, 2021
Sign up for freeto 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.

2 participants

@lobsterkatie@kamilogorek