equinor / fmu-editor

5 stars 2 forks source link

Upgrade electron and remove electron-msal #81

Closed mferrera closed 1 year ago

mferrera commented 1 year ago

Resolves #80

When removing electron-msal I tried to preserve files that are likely to be used again and as a result commented out codes remains. I removed code and packages that I believe won't be used or needed once the switch to azure/msal-react and azure/msal-browser happens. When a user clicks the logged-in silhouette they will see a notification that logging in is not currently supported.

Glancing at how msal-react works it seemed that "logged in" state management might differ so I removed the hook. It's not clear to me if interacting with the Microsoft user graph will differ.