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

Implement React GUI #69

Draft
wants to merge 89 commits into
base: main
Choose a base branch
from
Draft

Implement React GUI #69

wants to merge 89 commits into from

Conversation

inbarbarkai
Copy link
Collaborator

@inbarbarkai inbarbarkai commented Apr 27, 2022

  • Created a custom Keycloak image.
  • Implemented authentication flow.
  • Fix Keycloak seed.
  • Inviter:
    • Implement create invitation.
    • Implement update invitation.
  • Guard:
    • Implement active invitations screen.
    • Implement admit functionality.
  • Admin
    • Implement users screen.
    • Implement getting event logs.
    • Implement update user infomration.
    • Implement set user role.

Copy link
Contributor

@yarons yarons left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good.

docker-compose.yml Show resolved Hide resolved
web/src/services/authenticationService.ts Show resolved Hide resolved
@inbarbarkai
Copy link
Collaborator Author

inbarbarkai commented May 19, 2022

OK, the basic logic is in place.
@yarons Should I wait with this PR until the entire UI is implemented or should we merge?

@yarons
Copy link
Contributor

yarons commented May 19, 2022

@inbarbarkai we can wait for the UI, there's no other work done in other branches anyway.

Thank you so much!

inbarbarkai and others added 30 commits June 16, 2022 21:41
The following vulnerabilities are fixed with an upgrade:
- https://snyk.io/vuln/SNYK-ALPINE318-OPENSSL-6032386
- https://snyk.io/vuln/SNYK-ALPINE318-OPENSSL-6032386

Co-authored-by: snyk-bot <snyk-bot@snyk.io>
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.

None yet

4 participants