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

Mlucas/jobs UI copy code #249

Closed

Conversation

marjo-luc
Copy link
Member

No description provided.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@marjo-luc marjo-luc requested a review from rtapella June 21, 2023 21:09
Copy link
Collaborator

@rtapella rtapella left a comment

Choose a reason for hiding this comment

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

it works!

could probably use some built-in help or other indication of what that button does...

@rtapella
Copy link
Collaborator

rtapella commented Jun 22, 2023

hm in develop maap-documentation this should be in
system_reference_guide/ade_custom_extensions/jobsui.html

@rtapella
Copy link
Collaborator

oh this is just the docs 😆

@wildintellect wildintellect mentioned this pull request Jul 18, 2023
9 tasks
@wildintellect
Copy link
Collaborator

@marjo-luc @rtapella this didn't get merged and now has a merge conflict, who wants to resolve so we can merge?

@rtapella
Copy link
Collaborator

working...

rtapella added a commit that referenced this pull request Jul 18, 2023
- stuff from #249
- updated images showing the "Copy Jupyter Notebook Code" button during Job submission
- updated text describing that button
@rtapella
Copy link
Collaborator

I am re-building this in a new PR: #266

@rtapella
Copy link
Collaborator

I think one issue with this old branch is that it doesn't reflect the doc re-org for some reason...

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