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

Fixes #739 and fixes #740 by adding website to projects #741

Merged
merged 1 commit into from
Mar 9, 2017

Conversation

joshsmith
Copy link
Contributor

What's in this PR?

Adds website and should_link_externally columns to Project and exposes them on the view.

@ignu
Copy link
Contributor

ignu commented Mar 8, 2017

LGTM! 🚢 :shipit:

My only feedback is, as someone new to the project, I'm not sure what should_link_externally is for, even after reading the ticket. Is it just opening the project url in a new window?

@joshsmith
Copy link
Contributor Author

@ignu it's going to be very very temporary, but good feedback nonetheless.

@joshsmith joshsmith removed the request for review from christopherstyles March 9, 2017 00:13
@joshsmith joshsmith merged commit 542b38d into develop Mar 9, 2017
@joshsmith joshsmith deleted the 739-add-website-field branch March 9, 2017 00:17
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.

None yet

2 participants