-
Notifications
You must be signed in to change notification settings - Fork 44
Workflows #113
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
base: main
Are you sure you want to change the base?
Workflows #113
Conversation
|
@arthaud could you please review the changes |
|
LGTM but it seems like you upgraded the react version in sapp/ui/frontend/package-lock.json so I need to double check that the UI still works. |
|
sure if there any other input required from my side, do tell! |
|
Glad I tried. This is indeed breaking the UI. Running Could you please revert your changes to sapp/ui/frontend/package.json and sapp/ui/frontend/package-lock.json? Thanks for contributing :) |
Pre-submission checklist
black .usort format .flake8Summary
Failed tests were the motivating factor to debug the problem.