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.

Sign up for GitHub

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

Jump to bottom

Change "Refresh" to "Reharvest" on harvest source pages #25

Closed
johnmartin opened this issue Mar 21, 2013 · 5 comments
Closed

Change "Refresh" to "Reharvest" on harvest source pages #25

johnmartin opened this issue Mar 21, 2013 · 5 comments

Comments

@johnmartin
Copy link
Contributor

Also should have confirmation dialog to say that it's a big job.
@ghost ghost assigned johnmartin Mar 21, 2013
@amercader
Copy link
Member

Nope

@johnmartin
Copy link
Contributor Author

@amercader FYI: I just changed the title of the issue... from "Reindex" to "Reharvest"

@shevski
Copy link

shevski commented Mar 21, 2013

@amercader What do you mean 'nope'?
You did not reply to Friday's email.

@amercader
Copy link
Member

"Reindex" has a very specific meaning in CKAN, which is updating the Solr search index. "Reharvest" is better than "Reindex". I still prefer "Refresh" but can live with "Reharvest" if that is so confusing.

@johnmartin
Copy link
Contributor Author

OK cool, my fault with some confusion here as I misread the original email.

johnmartin added a commit that referenced this issue Mar 21, 2013
@johnmartin
[#25] Changed 'Refresh' to 'Reharvest' on button
3197162
johnmartin added a commit that referenced this issue Mar 21, 2013
@johnmartin
[#25] Adds confirmation dialog to reharvest button
78bde97
@johnmartin johnmartin mentioned this issue Mar 21, 2013
Merged
johnmartin added a commit that referenced this issue Mar 21, 2013
@johnmartin
[#25] Small copy tweak to confirmation dialog
4d0dd9a
@johnmartin johnmartin mentioned this issue Mar 21, 2013
Merged
amercader added a commit that referenced this issue Mar 21, 2013
@amercader
[#25] Further tweaks on helpers texts
7bff041
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@johnmartin @amercader @shevski