-
Notifications
You must be signed in to change notification settings - Fork 46
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
webconsole refactoring with react 18 #47
Conversation
frontend/WebUI-diff/WebUI.diff
Outdated
@@ -0,0 +1,234 @@ | |||
diff --color -rc webconsole-orig/backend/WebUI/api_webui.go webconsole/backend/WebUI/api_webui.go |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @kishiguro
Do we need this patch file after PR merged?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ianchen0119 No, no need of merging the patch. it is included in the PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ianchen0119 diff has been removed to avoid confusion.
@kishiguro |
@brianchennn are you able to login with User: free5gc Pass: admin ? |
Yes, I can login now. User: admin. Password: free5gc |
There are some issues:
|
nice!
Let me double check with the same configuration |
@kishiguro
|
@kishiguro |
@kishiguro |
@brianchennn just fixed a issue of admin tenant handling. now it should show subscription which created before. |
@brianchennn Seems there is a bug of setting flow rules. Let me fix it. |
@brianchennn FlowRule (and QosFlows) handling has been fixed. Would you mind to try again? |
@kishiguro |
Thanks for pointing out. Just fixed the issue. Would you mind to check it out? |
@kishiguro |
sure, wait a sec. |
@brianchennn just added support for multiple UEs by increment SUPI. would you mind to try? |
@kishiguro It works well ! |
Now default S-NSSAI configuartion will be set. Would you mind to double check? |
@kishiguro I also found that when creating a tenant, the hint of this field is Tenant ID, but it should be Tenant Name. Tenant ID is random generated as UUID. |
@brianchennn got it let me integrate the confirm password feature. |
@brianchennn I have integrated Password Confirmation feature. "Tenant ID" is also fixed as "Tenant Name". |
@tim-ywliu |
webconsole refactoring with react 18 and typescript and material ui.