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

Status broken after domain change #14

Closed
jasonm opened this issue Jan 18, 2013 · 4 comments
Closed

Status broken after domain change #14

jasonm opened this issue Jan 18, 2013 · 4 comments

Comments

@jasonm
Copy link
Member

jasonm commented Jan 18, 2013

When changing domains from clahub.herokuapp.com to www.clahub.com, I neglected to update a production config variable that determines the hostname for GitHub's post-receive webhook notifications. As such, commits were not being updated as pass/fail based on CLA signature status.

This has been fixed and I updated all existing webhook URLs via API in production and verified this.

For folks with CLAs on the production site: I'd generally avoid publicizing user information, but I'd rather notify folks about this. I've emailed everyone whose email I can ascertain. For others I only have the GitHub username, and can't figure out how to send a GH message (didn't that feature exist previously?) As such, cc @maxandersen @webmink @pidster.

@jasonm
Copy link
Member Author

jasonm commented Jan 18, 2013

This is fixed as of Thu Jan 17 20:47:13 PST 2013

@jasonm jasonm closed this as completed Jan 18, 2013
@jasonm
Copy link
Member Author

jasonm commented Jan 18, 2013

P.S. The fix above was to configure the HOST environment variable on Heroku. Then I retroactively updated all webhook URLs on your repos with Agreement.all.each { |a| a.delete_github_repo_hook ; a.create_github_repo_hook }.

I realized I also needed to re-check all open Pull Requests, so I have now also done that with: Agreement.all.each { |a| a.check_open_pulls }

Hopefully all is now well.

@jasonm
Copy link
Member Author

jasonm commented Jan 18, 2013

So, as of Thu Jan 17 20:51:35 PST 2013

@jasonm
Copy link
Member Author

jasonm commented Jan 18, 2013

Pulls which are good to merge should look like: jasonm/clahub-test2#1

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

No branches or pull requests

1 participant