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

Addresses Issue 70 - Vancouver Projects #72

Merged
merged 4 commits into from Nov 13, 2018

Conversation

Projects
3 participants
@SamFritz
Member

SamFritz commented Nov 13, 2018

Resolves issue 70

This pull request updates the Vancouver datathon page to include team projects and removes trip planning information (no longer necessary).

Commit also includes minor clean up of spacing on GWU event page. Major updates for GWU will be included in a separate ticket and PR

Just needs a quick look over and we'll be good to publish.

@SamFritz SamFritz self-assigned this Nov 13, 2018

@SamFritz SamFritz requested review from ruebot and ianmilligan1 Nov 13, 2018

@ianmilligan1

Just some minor tweaks and I think it's ready to go.

<tr>
<td><b>Team IDG</b>
<ul>
<li>Ben O'Brien, <em>(National Library of New Zealand)</em></li>

This comment has been minimized.

@ianmilligan1

ianmilligan1 Nov 13, 2018

Member

I think in the list format we can remove the trailing , between name and institution

<td><b>Team IDG</b>
<ul>
<li>Ben O'Brien, <em>(National Library of New Zealand)</em></li>
<li>Carl Cooper, <em>(Bodleian Libraries - Oxford University)</em></li>

This comment has been minimized.

@ianmilligan1

ianmilligan1 Nov 13, 2018

Member

same here (and note double space after ,)

<tr>
<td><b>Team BC 2017 Politics</b>
<ul>
<li>Matt Huculak, <em>(University of Victoria)</em></li>

This comment has been minimized.

@ianmilligan1

ianmilligan1 Nov 13, 2018

Member

remove trailing commas in the rest of the teams too?

@ianmilligan1

This comment has been minimized.

Member

ianmilligan1 commented Nov 13, 2018

Looks good to me! If it looks good to you @ruebot I can merge + deploy.

@ruebot

ruebot approved these changes Nov 13, 2018

@ianmilligan1 ianmilligan1 merged commit 31a3b3c into master Nov 13, 2018

@ianmilligan1 ianmilligan1 deleted the issue-70 branch Nov 13, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment