Skip to content
This repository has been archived by the owner on Dec 8, 2017. It is now read-only.

elaborates directions to give a sense of timing #210

Merged
merged 2 commits into from
Jun 13, 2015
Merged

Conversation

gbinal
Copy link
Member

@gbinal gbinal commented Jun 2, 2015

Gives a bit of context so that folks understand that the initial merge isn't when /dashboard is updated. cc @gboone

@gbinal gbinal mentioned this pull request Jun 2, 2015
@mbland mbland mentioned this pull request Jun 2, 2015
@mbland
Copy link
Contributor

mbland commented Jun 2, 2015

OK by me, though I'd be OK adding something about:

  • updating the public Hub to publish the new/changed data
  • importing the data via _data/import-public.rb
  • pushing the updated data to GitHub

Though, longer term, we may be able to do better than this as hinted at in 18F/hub#306.

@gbinal
Copy link
Member Author

gbinal commented Jun 12, 2015

I modified the edit so that it more reasonably sets expectations - can take days, ask for help if you need it faster.

Since it's just directions in the readme and I'm wary of waiting for this workflow to be addressed, I'm going to go ahead and merge this in a few minutes.

gbinal added a commit that referenced this pull request Jun 13, 2015
elaborates directions to give a sense of timing
@gbinal gbinal merged commit f56873a into staging Jun 13, 2015
@gbinal gbinal deleted the edits-directions branch June 13, 2015 03:21
@gboone
Copy link
Contributor

gboone commented Jun 13, 2015

👍

  • Greg Boone, 18F

On Jun 12, 2015, at 2:30 PM, Gray Brooks notifications@github.com wrote:

I modified the edit so that it more reasonably sets expectations - can take
days, ask for help if you need it faster.

Since it's just directions in the readme and I'm wary of waiting for this
workflow to be addressed, I'm going to go ahead and merge this in a few
minutes.


Reply to this email directly or view it on GitHub
#210 (comment).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants