| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
|
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it. What to do if you already signed the CLAIndividual signers
Corporate signers
|
Sorry, something went wrong.
There was a problem hiding this comment.
Thanks for doing this @dominikfoldi. I think you have all the basic elements. The details need a bit of polishing.
Sorry, something went wrong.
…t to string during UserRecord serialization
|
CLAs look good, thanks! |
Sorry, something went wrong.
There was a problem hiding this comment.
Thanks @dominikfoldi. This is looking pretty good. I've pointed out few changes. While you're working on them. I'm going to start our internal API review process for this, so we can get it merged and released soon.
Sorry, something went wrong.
…rver to SDK error message mapping
… tests and add error handling if the response of the update API contains incorrect data
…moved unused parts of the tests, simplify claims initialization for TooLargeClaimsPayload test and introduced unit tests for UpdateUserAsync by mocking the httpclient
|
@hiranya911 I think I am resolved all your requests. Thanks to you I found a bug in the FirebaseUserManager's PostAsync method when the server responds with empty uid, I don't know if could ever happen but I covered that just in case. |
Sorry, something went wrong.
There was a problem hiding this comment.
Just 2 more comments. I also have submitted this API for internal review. I should hear back in a couple of days. Hopefully we can get this released next week.
Sorry, something went wrong.
…e strings on exception
There was a problem hiding this comment.
LGTM. Waiting API review approval for merge.
Sorry, something went wrong.
|
Thank you for this super fast process! Looking forward to it! |
Sorry, something went wrong.
|
@dominikfoldi this is released: https://firebase.google.com/support/release-notes/admin/dotnet#1.1.0 Hope you continue to contribute. |
Sorry, something went wrong.
|
@hiranya911 thank you for the release! Our next step is to integrate Firebase Authentication into our project. But after I am sure that I will contribute more! |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
I have implemented the Custom User Claims feature. I do not checked out the contributing guidelines yet but I want to get feedback from you about my changes. I will implement the tests after everything else looks good.
My main guideline was the Java SDK.
@hiranya911 could you please check this out?