-
Notifications
You must be signed in to change notification settings - Fork 35
DOCSP-44687 export to playground with Copilot #103
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
Conversation
✅ Deploy Preview for docs-mongodb-vscode ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
Thanks a ton @ltran-mdb2! LGTM with a couple minor style guide nits.
source/playgrounds.txt
Outdated
|
||
1. Open a project file. | ||
|
||
To open a file in |vscode-short|, click on the file name in 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.
(nit -- click)
You can usually drop on. There are a couple instances. I'll just mark this one.
To open a file in |vscode-short|, click on the file name in the | |
To open a file in |vscode-short|, click the file name in 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.
Done!
source/playgrounds.txt
Outdated
|
||
#. Highlight the code that you want to open in a playground. | ||
|
||
#. Right click to select :guilabel:`Export Code to Playground`. |
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.
(nit -- right-click)
Always hyphenate. There are a couple instances. I'll just mark this one.
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.
Done!
source/playgrounds.txt
Outdated
|
||
.. include:: /includes/steps/open-new-playground.rst | ||
|
||
.. tab:: Your Project File |
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.
@ltran-mdb2 I think this might best for this to live in it's own nav as it is a complimentary feature to "Export to Language". We could call it "Test Application Query" in the nav and make the title "Export and Test Application Query in Playgrounds".
Lastly, on both of them (i.e. Export to Language) - could we call out that these features are currently only available for users who have Github Copilot & MongoDB's Copilot extension enabled?
source/playgrounds.txt
Outdated
.. tab:: Your Project File | ||
:tabid: project-file | ||
|
||
1. Open a project file. |
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.
Needs to be more specific, something like:
"Open your application project file that has the MongoDB query you want to export"
source/playgrounds.txt
Outdated
|
||
1. Open a project file. | ||
|
||
To open a file in |vscode-short|, click the file name in 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.
I think we can omit this part (fairly obvious for users who use the IDE)
source/playgrounds.txt
Outdated
To open a file in |vscode-short|, click the file name in the | ||
file explorer. | ||
|
||
#. Highlight the code that you want to open in a playground. |
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.
Here we should be specific about "highlighting the code of the query you want to test"
source/playgrounds.txt
Outdated
#. Right-click to select :guilabel:`Export Code to Playground`. | ||
|
||
Right-click your selected code to open the options menu. | ||
Click :guilabel:`MongoDB Copilot Participant` > |
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.
In the immediate update coming up we're renaming this to "MongoDB Copilot Extension" instead of "Participant". Let's change it here so we don't have to amend shortly after.
source/playgrounds.txt
Outdated
:guilabel:`Export Code to Playground`. | ||
|
||
When you click :guilabel:`Export Code to Playground`, |vscode-short| | ||
opens a new playground with your selected code. |
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.
Let's specify that this opens a new MongoDB playground file with your query exported to MongoDB Shell syntax. You can then hit the run button in the top left corner to test this query with the cluster you are currently connected to in your MongoDB extension. You can iterate on your query in the playgrounds until you are happy with your query. Once the query looks good to you, you can export it back to your application code by using the Export to Language feature (link to Export to Language page).
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.
I will add the note about exporting to language with the Export to Language PR (since this one will be merged first)
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.
A couple of minor things!
source/index.txt
Outdated
Connect </connect> | ||
Manage Data </databases-collections> | ||
Explore with Playgrounds </playgrounds> | ||
Test Queries </copilot-export-test> |
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.
Sorry for the miscommunication here @ltran-mdb2 – I meant to put this under "Explore with Playgrounds" tab, same level as "Export to Language".
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.
Could we also include the word "Driver" in here? We can call it "Test Driver Queries"
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.
Done!
source/playgrounds.txt
Outdated
|
||
.. note:: | ||
|
||
You can also open a new playground to test and export queries with 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.
Could we also add this note at the top of the "Export to Language" page? That feature now exclusively requires our Copilot extension.
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.
Done!
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.
LGTM, thank you!
DESCRIPTION
Users can now export code from their project files to a playground.
STAGING
https://deploy-preview-103--docs-mongodb-vscode.netlify.app/playgrounds/#create-a-new-playground
https://deploy-preview-103--docs-mongodb-vscode.netlify.app/copilot-export-test/
https://deploy-preview-103--docs-mongodb-vscode.netlify.app/export-to-language/#prerequisites
JIRA
https://jira.mongodb.org/browse/DOCSP-44687
Self-Review Checklist
External Review Requirements
What's expected of an external reviewer?