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

Document JavaScript debugging #31

Closed
jmakeig opened this issue Apr 7, 2020 · 20 comments
Closed

Document JavaScript debugging #31

jmakeig opened this issue Apr 7, 2020 · 20 comments
Assignees
Milestone

Comments

@jmakeig
Copy link

jmakeig commented Apr 7, 2020

Update the README with a getting started guide quickly covering the “why” and the “how”, including the configuration options and a screenshot showing the debugger in action. Also include a link to MarkLogic documentation about required privileges.

@jmakeig jmakeig added this to the 2.0.0 milestone Apr 7, 2020
@zzzwan
Copy link
Contributor

zzzwan commented Apr 7, 2020

@jmckean17 Hi Jane, could you take a look of the doc at https://github.com/marklogic/mlxprs/blob/sjsdbg/client/JSDebugger/README.md

@jmakeig
Copy link
Author

jmakeig commented Apr 7, 2020

@zzzwan and @jmckean17, I think we need to merge that content into the main README. This what shows up at https://marketplace.visualstudio.com/items?itemName=mlxprs.mlxprs.

Pinging @mikrovvelle to make sure this is the right approach.

@mikrovvelle
Copy link
Contributor

mikrovvelle commented Apr 8, 2020

That's right, the top-level readme is the only one that you see on the marketplace.

If debugging instructions get too in-depth for one document, we could leave a brief summary of "getting started" debugging in the top readme, and then link to a dedicated one for debugging.

@zzzwan
Copy link
Contributor

zzzwan commented Apr 8, 2020

@jmakeig @mikrovvelle Do we have a decision of where to put the js debugger readme? Copy the content over to the main README or have a reference/link there?

@mikrovvelle
Copy link
Contributor

We could just assign this one to me. I’ll try to put a succinct summary in the top-level readme, and then decide if and how much content should go in the sjsdbg one. Any objections?

@zzzwan
Copy link
Contributor

zzzwan commented Apr 8, 2020

No objection

@mikrovvelle mikrovvelle assigned mikrovvelle and unassigned jmckean17 and zzzwan Apr 8, 2020
@mikrovvelle
Copy link
Contributor

Ok, thanks. May re-assign it back later when ready for review.

@jmckean17
Copy link
Contributor

I'm wrapping up doc bugs for the 10.0-4 server release and will be able to focus on this early next week. Let me know if that will be an issue. Thanks!

@mikrovvelle
Copy link
Contributor

Reässigned to @jmckean17 for review. On the sjsdbg branch: I've taken the debug setup documentation out of sjsdbg/readme and put it into our top-level readme. I tried to tighten it up a little too.

Please close the issue if it looks good, or Slack me if not.

@jmckean17
Copy link
Contributor

Publishing the 10.0-4 docs today or tomorrow, then I can focus on this. Thanks.

@jmakeig jmakeig mentioned this issue Apr 14, 2020
8 tasks
@jmckean17
Copy link
Contributor

Hi, the 10.0-4 docs are published and I can focus on this now. Can you send me the link to the content you want me to look at? It seems like there's more than one link. Thanks!

@jmckean17
Copy link
Contributor

Apologies in advance for the newbee question: What is the most efficient way for me to edit the README? I'm looking at the README here: https://github.com/mikrovvelle/mlxprs. Thanks!

@mikrovvelle
Copy link
Contributor

No worries. Go here: https://github.com/mikrovvelle/mlxprs/blob/sjsdbg/README.md
This is the README version that needs your review (it will replace the current one, which is where your link points). You should be able to edit it directly on the page—there's a pencil icon, upper right relative to the text.

@jmckean17
Copy link
Contributor

So when I have edited the copy, I click commit and create a pull request? Is that correct? I'm modifying the wording, so I assume you'll review when you review the pull request.

@mikrovvelle
Copy link
Contributor

Yes, click "commit". You don't need to create a pull request. I'll just review your edits on the sjsdbg branch.

@jmckean17
Copy link
Contributor

Do we have a screenshot of the debugger in action to add? And should we add a section on the required privileges? (What are the required privileges?) I'm not sure where we would want to like to the server docs. Suggestions?

@mikrovvelle
Copy link
Contributor

Privileges are documented: https://github.com/mikrovvelle/mlxprs/blob/sjsdbg/README.md#required-privileges-for-eval-and-debugging

We do have a screenshot describing the interactive "attach" debugging mode in https://github.com/mikrovvelle/mlxprs/blob/sjsdbg/README.md#attach. Is there a specific point we need to additional screenshots to get across?

@jmckean17
Copy link
Contributor

I've been looking at an earlier version of the debugger. D'oh! I didn't see the screenshot or the permissions. I will edit this version and then click commit. :-)

@jmckean17
Copy link
Contributor

I have committed my changes.

@mikrovvelle
Copy link
Contributor

Thanks, looks great!

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

No branches or pull requests

4 participants