-
Notifications
You must be signed in to change notification settings - Fork 8
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(GameScreen): add "follow my bot" button #89
feat(GameScreen): add "follow my bot" button #89
Conversation
added zoom out and zoom in feature for when following bot.
Resolved the conflicts and changed the follow button label. |
Fixed the error that I was getting on first load. |
@aldrin312, thank you for the update 🙌 I'll take another look later this week. |
…ngs#92) ## How does this PR impact the user? https://github.com/user-attachments/assets/89f97e3f-75d4-414b-b102-0c86fd24eda8 ## Description - Added `/api/state` to public paths to make current game publicly accessible. - Updated the login screen to display the login form on the left and the current game on the right (reused the code from `index.vue`). - Moved the login form to a separate component (`/components/LoginForm.vue`). - Increased the timeout for the login e2e tests to 60 seconds, as 30 seconds timeout was insufficient for the tests to complete. ## Limitations The game screen takes a couple of seconds to load. ## Checklist - [x] my PR is focused and contains one wholistic change - [x] I have added screenshots or screen recordings to show the changes
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.
Thank you! It works great! Let's clean it up a bit more and merge.
You or I can take a look at what's up with appRef
when you try to move the zoom-in logic into a watch
callback in a follow-up.
I'll try looking into the |
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.
Thank you!
68ee712
into
move-fast-and-break-things:main
How does this PR impact the user?
Description
Limitations
Checklist