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

[ez] Redirect users to our issues creation page when submitting feedback #1362

Merged
merged 2 commits into from
Feb 26, 2024

Conversation

rossdanlm
Copy link
Contributor

@rossdanlm rossdanlm commented Feb 26, 2024

[ez] Redirect users to our issues creation page when submitting feedback

Super simple for now. Tried at first to connect to Jonathan's rage functiona but that was defined in the editor package: #870

Test Plan

e614d3f1-3840-4396-920d-10c694fb1d87.mp4

Stack created with Sapling. Best reviewed with ReviewStack.

I also made this a command that you can get from the command palette too.

Next PR will do implementation, just wanted to keep it simple and display this here for now


## Test Plan

Dark Mode
<img width="991" alt="Screenshot 2024-02-26 at 15 50 32" src="https://github.com/lastmile-ai/aiconfig/assets/151060367/10658e28-4dbc-4475-9994-bb806620e878">

Light Mode

<img width="996" alt="Screenshot 2024-02-26 at 15 50 43" src="https://github.com/lastmile-ai/aiconfig/assets/151060367/e865c149-8f10-4aaf-bfb0-97d04d1619e9">

Command Palette

<img width="874" alt="Screenshot 2024-02-26 at 15 51 53" src="https://github.com/lastmile-ai/aiconfig/assets/151060367/92072c41-66ac-400b-86f4-5a80bcbb0a84">
Super simple for now. Tried at first to connect to Jonathan's rage functiona but that was defined in the editor package: #870

## Test Plan

https://github.com/lastmile-ai/aiconfig/assets/151060367/1ab905d0-dec8-4168-9a4c-995b7af0cd28
Copy link
Contributor

@rholinshead rholinshead left a comment

Choose a reason for hiding this comment

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

I think this is a good starting point but +1 to the TODO for adding the additional info like log

@rossdanlm
Copy link
Contributor Author

I think this is a good starting point but +1 to the TODO for adding the additional info like log

I agree, just wanted something quick and dirty for now! Can fix forward later!

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.

3 participants