Question Answer Plugin

plugin
210

Hi Sam, what happened?

211

Not sure, but @chilldark is saying it does not work on latest

1 Like
212

@chilldark Weā€™ve already discussed the issue with your setup in some length in the topic you linked to.

As @fzngagan told you in that topic back in October:

Iā€™m happy to change this, but weā€™re running this plugin on many of our instances (both development and production) without any issues. You can make a PR for this change, weā€™ll wait on other people to report the issue and if they do report the same issue, @pacharanero will merge it. Meanwhile, you can use your PR branch to avoid the error.

Please do not repost the same issue in multiple places just to attract attention.

@sam It is working fine on the latest Discourse. See for example: What has cities, but no houses; forests, but no trees; and water, but no fish? - Sandbox - Pavilion (which is running the latest Discourse).

8 Likes
213

Itā€™s been working for us as well on latest.

1 Like
214

Hello,

We have an issue with this plugin. Some topics give 500 error. Log file says:

Started GET ā€œ/t/20390.jsonā€ for 81.215.15.180 at 2020-01-14 06:20:45 +0000
Processing by TopicsController#show as JSON
Parameters: {ā€œidā€=>ā€œ20390ā€}
Completed 500 Internal Server Error in 161ms (ActiveRecord: 0.0ms | Allocations: 35238)
NoMethodError (undefined method to_i' for [8, 8]:Array) /var/www/discourse/plugins/discourse-question-answer/lib/qa_post_edits.rb:115:in qa_vote_countā€™

1 Like
215

Thanks for the report. Iā€™ll take a look at it on Friday.

1 Like
216

Iā€™ve added some extra protections here to avoid the error youā€™re getting (which is specific to your instance). It should work for you now.

https://github.com/paviliondev/discourse-question-answer/commit/d7f2f9978d5095a22d8432c2aaa9bae3aa1c4d6a

2 Likes
217

Hello our issue fixed with this update thanks.

2 Likes
218

Hello,
I have the following warning in the devtools:

_ember_jquery-c2cca4a19fd4c70eabc95b010a6d7643955ec2a2251aeb1d70f625773eebea21.js:72538 [DEPRECATION] ember-addons/ember-computed-decorators is deprecated.
Please use discourse-common/utils/decorators instead.

I also have the same warning when I under the QA sandbox. I believe itā€™s due to the following line:

https://github.com/paviliondev/discourse-question-answer/blob/ae8d2e1ef65f1313e0e4ee9eef5cf4c4dc774635/assets/javascripts/discourse/initializers/qa-edits.js.es6#L2

Could it be it? If so, could it be possible to change it to discourse-common/utils/decorators as it suggested in the warning?

Also, for some reason, sometimes when user commented on an answer, it is not possible to comment on the first comment. For example I have the following structure:

  • Answer 1
    • Comment 1

Then, under comment 1 I donā€™t have the comment button. How should I comment on the comment 1? I do have the button comment under Answer 1 but it will not sort the comments by the date - the new comment will be above the first comment:

  • Answer 1
    • Comment 2
    • Comment 1
1 Like
219

Yes. I just updated Topic List Previews for this. It may take a while before a developer can attend to this. PR accepted

220

Uploaded a PR here.
Not sure how to solve my second question though.

1 Like
221

Thanks @Varsii, appreciated! We will get someone to review and merge the PRs (decorators one looks good, but Iā€™ll leave it for the assigned person).

@angus

222

Hello @angus, and thanks for this plugin! Your ā€œQ/A categoryā€ link above is broken. I would love to see this plugin in action. Do you have any other links to see this plugin live, in the wild?

223

@jord8on you have it here

3 Likes
224

Sorry I missed this, and THANK YOU for the direct link.

@angus just a thought - Several other plugins offer a link in the main description of the plugin, to ā€œsee the plugin in actionā€ Any chance you could add that link into the main topic description, here? Just a thought.

1 Like
225

Thanks for the note. Iā€™ve added an example link in the first post :+1:

3 Likes
226

Angus, you are a great Plugin Author. I have been following your posts.
I found a problem, this plugin is not compatible with some themes.

testplugin
testplugin869Ɨ880 44.4 KB

227

Or equally you might say that that Theme does not consider this plugin?

Is that Theme overriding a template? If so that may be part of the issue. The plugins we support target the base install and if a Theme works thatā€™s a pleasant benefit. I would extract the elements of that Theme that work and create your own.

3 Likes
228

Iā€™m now maintaner of this plugin and here is a quick update regarding our update policy:

How to safely update when using our plugins:

Pavilionā€™s update schedule

Pavilion will focus on ensuring compatibility of this plugin with Discourse Core tests-passed branch during the first 5 days of every month.

Any bugs which arise because of an incompatibility will be addressed asap during this ā€˜supported periodā€™, preferably by the 7th day. Low severity bugs or Beta features may be kept out of supported scope.

How to perform your update

Consider testing your update on your own staging server built from a back-up of your production site. You can switch this server off for the rest of the month so you donā€™t get charged for up-time. Use this opportunity to test out all basic functionality to identify any issues with plugins. Report all critical and high issues asap on the relevant plugin topic.

Aim to update your Production Discourse instance in the first 5 days of every month. You will get more focussed support from the Pavilion team on any issues with their plugins during this period.

You update your Production server outside of the first 5 days of the month at your own risk.

PS:
Major code rewrite is coming. You can follow it here:

https://github.com/paviliondev/discourse-question-answer/pull/30

4 Likes
229

Pardon the confusion: Iā€™m reading I update by the 5th day. Are you saying additional bug fixes will be released through the seventh day?

1 Like