follow-on to: #366722: Check for search service status in XMLRPC data

Seems some users are confused by Acquia Search being disabled and think it's a dependency problem. We should improve the xml-rpc checking too if possible.

CommentFileSizeAuthor
#7 fixup-398398-7.patch4.49 KBpwolanin
#6 fixup-398398-6.patch4.55 KBpwolanin
#5 fixup-398398-5.patch4.61 KBpwolanin
#3 better-wording-398398-3.patch3.65 KBpwolanin
#2 better-wording-398398-2.patch3.95 KBpwolanin
#1 better-wording-398398-1.patch2.17 KBpwolanin

Comments

pwolanin’s picture

pwolanin commented
StatusFileSize
new better-wording-398398-1.patch2.17 KB

Jacob's patch - I think it's incomplete.

pwolanin’s picture

pwolanin commented
Status: Active » Needs review
StatusFileSize
new better-wording-398398-2.patch3.95 KB

More complete patch - but depends on us making a minor change in acquia.com

pwolanin’s picture

pwolanin commented
StatusFileSize
new better-wording-398398-3.patch3.65 KB

Fix typo.

JacobSingh’s picture

JacobSingh commented
Status: Needs review » Closed (fixed)

Since I already had committed the patch above, I just committed an update to this as opposed to using this issue.

@pwolanin: If you feel called ot use the issue queue instead of CVS, you can revert and I can generate patches, but I figured since we're the only one's developing it, might as well just use a VCS. I'm closing this for now.

I managed to kludge in what Jeff wanted, he didn't like the solution above.

pwolanin’s picture

pwolanin commented
Status: Closed (fixed) » Needs review
StatusFileSize
new fixup-398398-5.patch4.61 KB

fix links and change text along lines suggested by Jeff.

pwolanin’s picture

pwolanin commented
StatusFileSize
new fixup-398398-6.patch4.55 KB

fewer words

pwolanin’s picture

pwolanin commented
Status: Needs review » Fixed
StatusFileSize
new fixup-398398-7.patch4.49 KB

final version

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.