Skip to content

[ENH] Starter Kit dropdown menu#278

Merged
effigies merged 4 commits intobids-standard:masterfrom
franklin-feingold:dropdown_menu
Jul 25, 2019
Merged

[ENH] Starter Kit dropdown menu#278
effigies merged 4 commits intobids-standard:masterfrom
franklin-feingold:dropdown_menu

Conversation

@franklin-feingold
Copy link
Copy Markdown
Collaborator

@franklin-feingold franklin-feingold commented Jul 18, 2019

Added direct links to - starter kit GitHub repo, tutorials, forum, and wiki. Here is the rendering from the index.

This can bring more attention to the starter kit by directly linking to these resources.

dorahermes
dorahermes previously approved these changes Jul 19, 2019
Copy link
Copy Markdown
Member

@dorahermes dorahermes left a comment

Choose a reason for hiding this comment

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

thanks! lgtm

sappelhoff
sappelhoff previously approved these changes Jul 19, 2019
Copy link
Copy Markdown
Member

@sappelhoff sappelhoff left a comment

Choose a reason for hiding this comment

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

As discussed earlier, I think this keeps the balance between not cluttering the specification yet providing some handles for quickly going to tutorials.

Edit: Forgot to say thanks. Thank you Franklin :-)

@sappelhoff sappelhoff added this to the 1.2.1 milestone Jul 22, 2019
@sappelhoff sappelhoff dismissed their stale review July 23, 2019 09:04

need to finish discussion first: #278 (comment)

Co-Authored-By: Stefan Appelhoff <stefan.appelhoff@mailbox.org>
@franklin-feingold franklin-feingold dismissed stale reviews from dorahermes via d9c7087 July 24, 2019 19:33
@effigies effigies merged commit a58a389 into bids-standard:master Jul 25, 2019
@franklin-feingold franklin-feingold deleted the dropdown_menu branch August 5, 2019 18:53
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.

4 participants