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

[Google Form] New FAQ! #5487

Merged
merged 14 commits into from
Oct 30, 2024
Merged

[Google Form] New FAQ! #5487

merged 14 commits into from
Oct 30, 2024

Conversation

github-actions[bot]
Copy link
Contributor

Automated changes by create-pull-request GitHub action

Copy link
Member

@hexylena hexylena left a comment

Choose a reason for hiding this comment

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

Sorry @hexhowells @nomadscientist our parser had | as a quotechar so when @nomadscientist used it for the first time it broke parsing 😅 thanks for testing our code paths. It should never have been set as the quotechar.

@hexhowells
Copy link
Collaborator

I'm not able to make changes here but the formatting of the numbered list seems a bit broke as most of the list items are on the same line. I also accidently submitted a slightly older version of this, the last line should be modified to the following:

9. Since this list is for `wget` commands, you need to extract out the links from the command. You can copy the link in the first set of double quotes for each line and {% icon galaxy-wf-edit %} **Paste/Fetch Data** them directly into Galaxy to download the files.

I'm happy to make the changes myself if given access!

@hexhowells
Copy link
Collaborator

Those changes are for importing-data-from-sierra-lims.md to be clear

@hexylena
Copy link
Member

Invite sent @hexhowells

@hexylena hexylena merged commit 3b04f69 into main Oct 30, 2024
4 checks passed
@hexylena hexylena deleted the create-pull-request/patch-1730275365 branch November 7, 2024 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants