-
Notifications
You must be signed in to change notification settings - Fork 1
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: ui upgrade #4
Conversation
added triple display
/review |
! Error: This model's maximum context length is 16385 tokens. However, your messages resulted in 85424 tokens. Please reduce the length of the messages. |
/review |
Have achieved specificationThe changes in the PR align with the specification provided. Here is a summary of the changes made:
Please review the implementation in more detail to ensure it meets all requirements and functions as expected. Ensure the pull request requirements are in the linked issue's first comment and update it if the scope evolves.
|
/review |
Have achieved specificationThe changes in the PR align with the specification provided. Here is a summary of the changes made:
Please review the implementation in more detail to ensure it meets all requirements and functions as expected. Ensure the pull request requirements are in the linked issue's first comment and update it if the scope evolves. |
/review |
Haven't achieved specificationThe answer provided contains unnecessary information and is not formatted correctly. Here's how you can fix it:
Example of a corrected answer:
Please make these changes and resubmit your answer. Ensure the pull request requirements are in the linked issue's first comment and update it if the scope evolves. |
/review |
Haven't achieved specificationThe answer provided contains unnecessary information and is not formatted correctly. Here's how you can fix it:
Here's an example of how the answer should be formatted:
Please make these changes and resubmit your answer. Ensure the pull request requirements are in the linked issue's first comment and update it if the scope evolves. |
/review |
Haven't achieved specificationThe answer provided contains unnecessary information and is not formatted correctly. Here's how you can fix it:
Example of a corrected answer:
Please make the necessary changes and resubmit your answer. Ensure the pull request requirements are in the linked issue's first comment and update it if the scope evolves. |
/review |
Have achieved specificationBased on the PR diff, it appears that the specification has been met.
Overall, the changes in the PR align with the specification provided. Ensure the pull request requirements are in the linked issue's first comment and update it if the scope evolves. |
/review |
Have achieved specificationThe changes in the PR align with the specification provided. Ensure the pull request requirements are in the linked issue's first comment and update it if the scope evolves.
|
/review |
Have achieved specificationBased on the PR diff, it appears that the specification has been met.
Overall, the PR implements the required layout and components for the Chat, Data, and Docs pages. Ensure the pull request requirements are in the linked issue's first comment and update it if the scope evolves.
|
/review |
Have achieved specificationBased on the PR diff, it appears that the specification has been met.
Ensure the pull request requirements are in the linked issue's first comment and update it if the scope evolves. |
/review |
Have achieved specificationBased on the PR diff, it appears that the specification has been met.
Overall, the PR implements the required layout and components for the Chat, Data, and Docs pages. Ensure the pull request requirements are in the linked issue's first comment and update it if the scope evolves. |
/review |
Spec Achieved: Based on the PR diff, it appears that the specification has been met.
Overall, the PR implements the required layout and components for the Chat, Data, and Docs pages. Ensure the pull request requirements are in the linked issue's first comment and update it if the scope evolves. |
/review |
Spec Achieved: Have achieved specification{username}, you have achieved the spec and now the reviewers will let you know if there are any other changes needed. Ensure the pull request requirements are in the linked issue's first comment and update it if the scope evolves.
|
/review |
Spec Achieved: Have achieved specificationBased on the PR diff, it appears that the specification has been met.
Overall, the PR implements the required layout and components for the Chat, Data, and Docs pages. Ensure the pull request requirements are in the linked issue's first comment and update it if the scope evolves. |
/review |
Spec Achieved: Have achieved specificationEnsure the pull request requirements are in the linked issue's first comment and update it if the scope evolves. |
/review |
Spec achievedYou have achieved the spec and now the reviewers will let you know if there are any other changes needed. Ensure the pull request requirements are in the linked issue's first comment and update it if the scope evolves.
|
/review |
Spec achievedKeyrxng, you have achieved the spec and now the reviewers will let you know if there are any other changes needed. Ensure the pull request requirements are in the linked issue's first comment and update it if the scope evolves.
|
/review |
Spec achievedKeyrxng, you have achieved the spec and now the reviewers will let you know if there are any other changes needed. Ensure the pull request requirements are in the linked issue's first comment and update it if the scope evolves.
|
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.
testingggg
Resolves #3
I went ahead and used NextJs routing to implement things.
The displays are aligned so that the data is on the left, chat displayed in the middle and docs on the right all in parallel.