Change Provider enum to ProviderEnum in order to avoid name conflicts with riverpod

[supabase]

Change Provider enum to ProviderEnum in order to avoid name conflicts with riverpod

[igoriuz]

Thanks for your fast reply @dshukertjr

Alternatively developers can hide this class on either of these imports.. for e.g.

import 'package:supabase/supabase.dart' hide Provider

My last suggestion would be renaming it to OAuthProvider but i can live with it.

[dshukertjr]

Hi @igoriuz !

Thank you so much for opening an issue and submitting a PR about naming conflict with Gotrue and Riverpod.

I understand your pain of having to use an alias every time you use Gotrue with Riverpod.

I found an interesting thread of naming conflicts between Flutter and another package. We did chat about this issue internally, but unfortunately naming conflict could occur between any packages, and we believe it is better to have solid intuitive API than to avoid naming conflicts and end up with none-ideal names. We just cannot keep changing symbol names every time there is a conflict with other packages.

So in order to use Riverpod and Gotrue together this would be the way to do it:

import 'package:gotrue/gotrue.dart' as gotrue;

final res = await gotrueClient.signIn(provider: gotrue.Provider.github);

Or with Supabase:

import 'package:supabase/supabase.dart' as supabase;

final res = await gotrueClient.signIn(provider: supabase.Provider.github);

I really do understand the pain, and this is probably is not the answer you wanted, but I hope you could understand. We appreciate your feedback, and please don't hesitate to let us know any thoughts that might come up in the future!