Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Ilmoitus käyttäjlle järjestelmätason oikeuksien puuttumisesta; Korjaa uloskirjautumista #609

Merged
merged 3 commits into from
Feb 21, 2023

Conversation

tkork
Copy link
Contributor

@tkork tkork commented Feb 17, 2023

No description provided.

@tkork tkork closed this Feb 17, 2023
@tkork tkork reopened this Feb 17, 2023
@tkork
Copy link
Contributor Author

tkork commented Feb 17, 2023

Rebasettelen tätä vähä. Tässä tuli ylimääräsiä committeja

@tkork tkork marked this pull request as ready for review February 17, 2023 10:16
@tkork
Copy link
Contributor Author

tkork commented Feb 17, 2023

Nyt ois valmis katselmointiin

@tkork
Copy link
Contributor Author

tkork commented Feb 17, 2023

Tuli mieleen, että kannattaisiko toi unauthorized state siirtää tohon _app-komponenttiin ja sen staten updateemiseen callback-funkkari annettais ApiProviderille propsina. Sit ei tarvis muokata näin montaa tiedostoa ja "turhaan" lisätä ApiProviderContextiin tota unauthorized-kenttää.

Kun toi unauthorized tieto on ainoastaan relevantti tässä toteutuksessa tossa _app-tiedostossa.

@kettunju kettunju self-assigned this Feb 20, 2023
@tkork tkork merged commit eddb413 into main Feb 21, 2023
@tkork tkork deleted the feature/HASSU-1491 branch February 21, 2023 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants