Skip to content

CORE-1889 Add Tapis v2 to v3 app migration guide - #3

Merged
psarando merged 3 commits into
cyverse:mainfrom
psarando:tapis-v3
Jun 28, 2024
Merged

CORE-1889 Add Tapis v2 to v3 app migration guide#3
psarando merged 3 commits into
cyverse:mainfrom
psarando:tapis-v3

Conversation

@psarando

Copy link
Copy Markdown
Member

Not sure if we need to add a link to this doc in the left Nav/TOC.

@psarando
psarando requested a review from slr71 June 14, 2024 02:41
Added notes about input/output paths inside job Docker containers.

@slr71 slr71 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looks good to me! 👍

Comment thread docs/services/api/tapis-v2-v3-migration.md Outdated
* Updated `agave://data.cyverse.rocks` to
`agave://data.iplantcollaborative.org` for consistency.
* Clarified line about `notes` fields that are not ignored by the DE.
* Clarified phrasing about `notes.semantics.ontology` lists.
@psarando

Copy link
Copy Markdown
Member Author

Thanks for the review! 👍

@psarando
psarando merged commit 93f27dc into cyverse:main Jun 28, 2024
@psarando
psarando deleted the tapis-v3 branch June 28, 2024 21:34
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.

2 participants