Skip to content

Adding "utmParametersDictionary" api to FIRDynamicLink #7505

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

Merged
merged 5 commits into from
Feb 18, 2021

Conversation

eldhosembabu
Copy link
Contributor

Adding "utmParametersDictionary" api to FIRDynamicLink.

This new property will enable 3p developers to fetch the Utm parameters associated with a Firebase Dynamic Link.

Related Issue: #6730

API Review Doc : https://docs.google.com/document/d/1NDBjiuCQEs3vnaPGfYKK2cdvJMIsJqukRtrZf8wsWCk (Approved)

@google-oss-bot
Copy link

google-oss-bot commented Feb 10, 2021

Coverage Report

Affected SDKs

  • FirebaseDynamicLinks-iOS-FirebaseDynamicLinks.framework

    SDK overall coverage changed from ? (f4a7b1a) to 76.69% (cd0a4b7) by ?.

    Click to show coverage changes in 14 files.
    Filename Base (f4a7b1a) Head (cd0a4b7) Diff
    FDLLogging.m ? 61.29% ?
    FDLURLComponents.m ? 82.49% ?
    FDLUtilities.m ? 97.35% ?
    FIRDLDefaultRetrievalProcessV2.m ? 73.96% ?
    FIRDLJavaScriptExecutor.m ? 85.22% ?
    FIRDLRetrievalProcessFactory.m ? 100.00% ?
    FIRDLRetrievalProcessResult.m ? 100.00% ?
    FIRDLScionLogging.m ? 100.00% ?
    FIRDynamicLink.m ? 81.90% ?
    FIRDynamicLinkComponentsKeyProvider.m ? 60.00% ?
    FIRDynamicLinkNetworking.m ? 81.67% ?
    FIRDynamicLinks.m ? 69.84% ?
    GINArgument.m ? 0.00% ?
    GINInvocation.m ? 0.00% ?

Test Logs

Copy link
Member

@paulb777 paulb777 left a comment

Choose a reason for hiding this comment

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

Generally looks good, two things to address ...

Copy link
Member

@paulb777 paulb777 left a comment

Choose a reason for hiding this comment

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

3 more nits and then should be good ...

Copy link
Contributor Author

@eldhosembabu eldhosembabu left a comment

Choose a reason for hiding this comment

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

Code review comments addressed

@eldhosembabu eldhosembabu merged commit 9efc2be into master Feb 18, 2021
@eldhosembabu eldhosembabu deleted the fdl-utm-params branch February 18, 2021 01:06
eldhosembabu added a commit to firebase/firebase-android-sdk that referenced this pull request Feb 18, 2021
Add getUtmParameters api method to PendingDynamicLinkData.

This is new api method to provide utm parameters associated with a firebase dynamic link.

Approved api review doc : https://docs.google.com/document/d/1NDBjiuCQEs3vnaPGfYKK2cdvJMIsJqukRtrZf8wsWCk

iOS PR for the same : firebase/firebase-ios-sdk#7505
ziadtamim pushed a commit to ziadtamim/firebase-ios-sdk that referenced this pull request Mar 11, 2021
Adding "utmParametersDictionary" api to FIRDynamicLink.

This new property will enable 3p developers to fetch the Utm parameters associated with a Firebase Dynamic Link.

Related Issue: firebase#6730

API Review Doc : https://docs.google.com/document/d/1NDBjiuCQEs3vnaPGfYKK2cdvJMIsJqukRtrZf8wsWCk (Approved)
@firebase firebase locked and limited conversation to collaborators Mar 21, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants