vue3-google-signin icon indicating copy to clipboard operation
vue3-google-signin copied to clipboard

Plans for Dynamic clientId Updates?

Open zuhno opened this issue 1 year ago • 10 comments

Hello :)

Is your feature request related to a problem? Please describe. I'm currently facing a limitation with the static assignment of the clientId value. My backend server provides redirect_uri and clientId, which I wish to dynamically utilize in my application. However, the current implementation does not allow for changing the clientId symbol value after initialization, leading to inflexibility in adapting to different server responses.

Describe the solution you'd like I would like the ability to dynamically update the clientId value. Specifically, the solution would involve:

  1. Using setClientId to change the clientId symbol value.
  2. Modifying useOneTap, useTokenClient, useCodeClient hooks to accept clientId as part of the options object, enabling the use of either the initialization value or the dynamically provided clientId.

To implement the above, it would be necessary to optionally accept clientId during app installation.

Describe alternatives you've considered An alternative could be a more static configuration, but this lacks the flexibility required for different backend responses. I'm open to other suggestions that could provide similar functionality.

Additional context The inability to dynamically update the clientId hinders the adaptability of the application to different backend environments. Also, there was an issue with the package.json version not being updated, causing a failure in GitHub Action and preventing the NPM version from updating. This is an additional context that might be relevant to the overall functionality and deployment process.

zuhno avatar Dec 30 '23 17:12 zuhno

Hi @zuhno this is a good suggestion, lets see what we can do here

im initially thinking of providing a method to update the client id value in the provider as you mentioned

kasvith avatar Dec 31 '23 06:12 kasvith

@kasvith If we develop using the methods provided by the provider, it seems necessary to verify the initialization of the clientId in the hooks. Having the hooks receive the clientId as an options member would have the advantage of allowing each hook to use a different clientId, though I haven't thought about it in detail yet.

Would it be okay for me to work on it as you suggested and submit a pull request?

zuhno avatar Dec 31 '23 09:12 zuhno

sure, go ahead and do a PR, lets review it

kasvith avatar Dec 31 '23 11:12 kasvith

Thanks @kasvith, Opened PR(https://github.com/wavezync/vue3-google-signin/pull/61). please review it.

zuhno avatar Jan 02 '24 16:01 zuhno

hi, will do. i got little busy

kasvith avatar Apr 23 '24 16:04 kasvith