Skip to content

feat(fcm): Set FCMOptions for MulticastMessage - #537

Merged
lahirumaramba merged 2 commits into
firebase:devfrom
FZambia:multicast_message_fcm_options
Mar 11, 2024
Merged

feat(fcm): Set FCMOptions for MulticastMessage#537
lahirumaramba merged 2 commits into
firebase:devfrom
FZambia:multicast_message_fcm_options

Conversation

@FZambia

@FZambia FZambia commented Feb 11, 2023

Copy link
Copy Markdown
Contributor

Hello!

Currently it's impossible to set FCMOptions for MulticastMessage. While for example in Python admin SDK this is possible. And in Java admin SDK. This pull request fixes this.

@google-cla

google-cla Bot commented Feb 11, 2023

Copy link
Copy Markdown

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@FZambia

FZambia commented Feb 15, 2023

Copy link
Copy Markdown
Contributor Author

CLA signed

@lahirumaramba

lahirumaramba commented Mar 24, 2023

Copy link
Copy Markdown
Member

Hi @FZambia Thank you for your contribution. Please change the base branch to dev before we can start the review process.

https://github.com/firebase/firebase-admin-go/blob/master/CONTRIBUTING.md#want-to-submit-a-pull-request

Make sure to create all your pull requests against the dev branch. All development work takes place on this branch, while the master branch is dedicated for released stable code. This enables us to review and merge routine code changes, without impacting downstream applications that are building against our master branch.

@FZambia
FZambia changed the base branch from master to dev March 24, 2023 17:14
@lahirumaramba lahirumaramba changed the title Possibility to set FCMOptions for MulticastMessage feat(fcm): Possibility to set FCMOptions for MulticastMessage Mar 28, 2023
@lahirumaramba lahirumaramba changed the title feat(fcm): Possibility to set FCMOptions for MulticastMessage feat(fcm): Set FCMOptions for MulticastMessage Mar 28, 2023
@FZambia

FZambia commented May 13, 2023

Copy link
Copy Markdown
Contributor Author

Hello team, could you please tell when this will be released?

@FZambia
FZambia force-pushed the multicast_message_fcm_options branch from 89ea601 to 8cb0fd4 Compare June 10, 2023 07:31
@FZambia

FZambia commented Jun 10, 2023

Copy link
Copy Markdown
Contributor Author

@lahirumaramba - any updates?

@FZambia

FZambia commented Dec 27, 2023

Copy link
Copy Markdown
Contributor Author

Guys, at least explain what prevents you from merging this?

@chong-shao

Copy link
Copy Markdown
Contributor

LGTM But I'd like to ask for waiting until @lahirumaramba 's review

@sebastianbuechler

Copy link
Copy Markdown

This would be nice to finally have in the package. @lahirumaramba can you please have a look at this minor change as it's only two lines of code and almost a year open?

@FZambia

FZambia commented Mar 3, 2024

Copy link
Copy Markdown
Contributor Author

@lahirumaramba kindly reminding 🙏

@lahirumaramba lahirumaramba left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!

@lahirumaramba
lahirumaramba merged commit ca81030 into firebase:dev Mar 11, 2024
@lahirumaramba

Copy link
Copy Markdown
Member

Thanks folks! This change will be included in the next release

@FZambia

FZambia commented Mar 12, 2024

Copy link
Copy Markdown
Contributor Author

Nice, many thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants