lil-lab / recnet

A human-driven recommendation system for academic readings.
https://recnet.io
MIT License
3 stars 1 forks source link

Firebase authentication migration #91

Closed anya-ji closed 8 months ago

anya-ji commented 9 months ago

Migrate Firebase project to the new org

yoavartzi commented 8 months ago

@anya-ji Maybe this should wait while @joannechen1223 is migrating most of the DB functionality to a postgres SQL server. I hope this will happen over the next few weeks. Then there's the issue of authentication, which is somehow also connected to Firebase, so I am not completely sure.

anya-ji commented 8 months ago

Yes, auth can be reconfigured by simply changing the config file to the new org's project configs. Another thing attached to the Cornell firebase account is email digests. It just needs a reconfiguration and redeployment of the cloud function on the new org's project console as well. Ofc I don't think we actually need to do these at all if these two functionalities are going to be moved away from firebase as well.

yoavartzi commented 8 months ago

Any idea how authentication migration is going to influence existing accounts? If it's going to effect them.

anya-ji commented 8 months ago

They might be logged out and just need to log in again. The recnet accounts won't be affected because they are stored in the database.

yoavartzi commented 8 months ago

@joannechen1223 can you look into it as part of the broader migration project? We need to continue using Google Authentication, but integrate it into our new backend, or at the least move it away from Anya's accounts after we migrate away from Firebase for everything else. Thanks!

anya-ji commented 8 months ago

I don't think anything is under my account, so anyone from Cornell added to this project can access/edit/own it. It's just that the project is under the Cornell organization which might be inconvenient in the long run.

yoavartzi commented 8 months ago

Right. I forgot that it's under the Cornell org. Thanks!

yoavartzi commented 8 months ago

Closing this issue for now. We will figure out authentication soon enough. Thanks!