Commit Graph

97 Commits

Author SHA1 Message Date
communiteq ca0dbd6d4c
Two extra options (#138)
* Make it possible to disallow topic starters to accept a solution

* Make adding QAPage schema markup optional

* Address @eviltrout's comments

Co-authored-by: Richard <richard@communiteq.com>
2021-06-17 16:12:33 -04:00
Mark VanLandingham 7351882171
Revert "Use `add_directory_column` API to add solutions count to user directory (#137)" (#139)
This reverts commit 58c594fdda.
2021-06-17 12:37:29 -05:00
Mark VanLandingham 58c594fdda
Use `add_directory_column` API to add solutions count to user directory (#137) 2021-06-17 09:31:11 -05:00
Arpit Jalan bbbd13de85
FEATURE: show education message in composer when replying on solved topic (#135) 2021-06-14 16:06:55 +05:30
Jarek Radosz ce63d80abb
DEV: Avoid nil custom fields (#128) 2021-05-05 12:19:00 +02:00
Martin Brennan c722b94a97
FIX: Update set_or_create_topic_timer to use duration_minutes keyword
See https://github.com/discourse/discourse/pull/11961
2021-02-05 15:33:09 +10:00
Bianca Nenciu 6d91419caf
FIX: Refresh whole post stream when category changes (#111)
Only if the old and new category have different 'solved' settings.
2021-01-20 12:26:37 +02:00
Krzysztof Kotlarek 5840601025
FEATURE: silently auto close solved topic (#104)
Topic solved should be closed silently as system user involvement may be confusing.

In addition, old rescue block was dropped.

Meta: https://meta.discourse.org/t/discourse-encrypt-unable-to-see-close-message/135235
2020-12-03 15:01:42 +11:00
Vinoth Kannan f04ccbd4e6
FEATURE: send notification to OP when staff marked a post as solution. (#97)
Previously, we won't send any notification to the original poster when a staff user marked a post as the solution on behalf of OP.
2020-10-30 22:01:27 +05:30
Régis Hanol 9ecac7a3e2 FIX: add mutex around un/mark as solved
To prevent any race conditions when two users toggle the state of a reply.

(cf. https://meta.discourse.org/161104)
2020-08-17 16:33:24 +02:00
jbrw 6f1cf5a779
FEATURE - Category group moderators can (un)accept solutions, if enabled (#89) 2020-07-27 17:42:42 -04:00
Krzysztof Kotlarek 36579b774f FIX: check if add_post_custom_fields_allowlister exisits 2020-07-27 12:04:28 +10:00
Krzysztof Kotlarek 57a8ec8a06 FIX: remove empty line 2020-07-15 09:44:32 +10:00
Krzysztof Kotlarek 10c31ffbd5 FIX: compatibility after allowlist rewording in Discourse 2020-07-15 09:34:02 +10:00
Krzysztof Kotlarek 922b4c3907
FIX: plugin backwards compatibility with Discourse 2.4 (#87)
Duration is a new keyword argument on Topic#set_or_create_timer and is not yet available in stable Discourse
2020-04-27 13:01:01 +10:00
Dan Ungureanu dc87d34594
FEATURE: Add include_subcategories report filter
Follow up to dafb1d7c7f.
2020-04-24 13:16:15 +03:00
Dan Ungureanu dafb1d7c7f
FIX: Include subcategories in report (#86) 2020-04-10 12:50:44 +03:00
Vinoth Kannan 329d8aff30 FIX: use the new duration attribute in `set_or_create_timer` method.
aad12822b7
2020-03-19 22:53:17 +05:30
Blake Erickson 8d883fba93
FEATURE: Publish WebHook event when solving/unsolving (#85)
* FEATURE: Publish WebHook event when solving/unsolving

This feature will publish a post edit webhook event whenever a solution
is accepted or unaccepted.

I went ahead and used the existing post-edit webhook because all the
post custom fields for the solved plugin are already included in the
post-edit serializer.

* Create Solved Event Webhook

This commit adds a solved event webhook that will only trigger when an
answer has been marked as accepted or unaccepted.

It uses 100 as the webhook ID. This way any new webhooks in core can
keep using lower numbers like 11, 12, 13, but plugins can use 101, 102,
etc.

* Removed functionality that was added to core

This [PR][1] to discourse core adds what what removed in this commit.

It is better to have this logic in core so that it is discoverable and
future webhooks won't end up accidentally using the same ID.

[1]: https://github.com/discourse/discourse/pull/9110

* UX: Add "solved" status filter in advanced search page.

And rename `in:solved` to `status:solved`.

* FEATURE: Publish WebHook event when solving/unsolving

This feature will publish a post edit webhook event whenever a solution
is accepted or unaccepted.

I went ahead and used the existing post-edit webhook because all the
post custom fields for the solved plugin are already included in the
post-edit serializer.

* Create Solved Event Webhook

This commit adds a solved event webhook that will only trigger when an
answer has been marked as accepted or unaccepted.

It uses 100 as the webhook ID. This way any new webhooks in core can
keep using lower numbers like 11, 12, 13, but plugins can use 101, 102,
etc.

* Removed functionality that was added to core

This [PR][1] to discourse core adds what what removed in this commit.

It is better to have this logic in core so that it is discoverable and
future webhooks won't end up accidentally using the same ID.

[1]: https://github.com/discourse/discourse/pull/9110

Co-authored-by: Vinoth Kannan <vinothkannan@vinkas.com>
2020-03-06 11:28:29 -07:00
Vinoth Kannan f4aea44db8 UX: Add "solved" status filter in advanced search page.
And rename `in:solved` to `status:solved`.
2020-03-06 08:57:38 +05:30
Sam Saffron e5b08c71c4
FIX: do not allow whispers to be accepted as answers
This also cleans up the guardian so it is easier to reason about
2020-02-23 19:16:31 +11:00
Krzysztof Kotlarek 230c2071ea
FEATURE: Add helpdesk and tech support badges (#84)
* FEATURE: Add helpdesk and tech support badges

When plugin is installed, we should create helpdesk and tech support
badges.

Both badges should be disabled by default
2020-02-05 11:14:12 +11:00
Vinoth Kannan 27da70052f FIX: keep quoted content in schema data if post excerpt is empty. 2020-01-04 19:25:42 +05:30
Sam Saffron 4c0545a827 DEV: lint files
This removes explicit return where not needed per rubocop rule
2019-12-10 12:01:54 +11:00
Joffrey JAFFEUX 15bc64bbc9 DEV: uses Discourse.redis instead of $redis 2019-12-03 10:49:58 +01:00
Vinoth Kannan 797d450b73 FIX: keep onebox body content in post excerpts for schema data. 2019-11-02 17:31:38 +05:30
romanrizzi 747a2e5167 DEV: Remove old backward-compatibility code 2019-10-09 15:56:55 -03:00
David Taylor 486f343b09 FIX: Include previously solved topics in auto-bump
Null values can happen when a post is removed as the solution. We already filter for NOT NULL in the topic_query and search filters. This commit adds the same logic to the auto-bump filter.
2019-08-06 11:19:33 +01:00
Sam Saffron 0fa0e7fd53 DEV: support running plugin without db
This is required for asset precompile
2019-06-13 16:33:03 +10:00
Guo Xiang Tan 75046f1d92 Add frozen string literal comment to files. 2019-05-13 11:04:27 +08:00
Bianca Nenciu 44e832813c FIX: Align checked icon in title. 2019-05-10 14:07:00 +03:00
Maja Komel db35674cc4 FIX: show solved icon on categories with featured topics page 2019-05-10 10:43:11 +02:00
Joffrey JAFFEUX 9f27caf39c FIX: adds support for new filter syntax 2019-05-01 22:24:09 +02:00
Joffrey JAFFEUX c6926f647b FIX: removes legacy code 2019-04-01 15:31:20 +02:00
Bianca Nenciu 3297d152a7 FIX: Staff users can accept posts in deleted topics. (#75) 2019-03-18 16:27:29 +01:00
Vinoth Kannan ca2d537665 PERF: Remove N+1 queries on search topic list item serializer. 2019-03-17 23:20:05 +05:30
Vinoth Kannan 8b8512e73e DEV: Use core method to preload category custom fields for site serializer 2019-03-16 23:27:31 +05:30
Vinoth Kannan 8beaa6c1d9 PERF: Remove N+1 queries on basic category serializer. 2019-03-15 18:53:22 +05:30
Vinoth Kannan 0fa41a8b38 PERF: Remove N+1 queries on site serializer. 2019-03-15 17:25:17 +05:30
Guo Xiang Tan 818bff2b30 FIX: Remove topic answer mixin from category detailed serializer. 2019-03-14 05:31:51 +08:00
Vinoth Kannan 5ddd644e00 FEATURE: Show dropdown to filter topic lists by solved status 2019-03-14 00:41:07 +05:30
Vinoth Kannan 6fb6f8849b Revert "REVERT: 'FEATURE: Add solved icon for topic titles in search results and other topic lists'"
This reverts commit 6dbd420c9e.
2019-03-13 22:40:54 +05:30
Vinoth Kannan 6dbd420c9e REVERT: 'FEATURE: Add solved icon for topic titles in search results and other topic lists' 2019-03-13 19:38:31 +05:30
Vinoth Kannan e9be43becd FEATURE: Add solved icon for topic titles in search results and other topic lists 2019-03-13 15:59:07 +05:30
Vinoth Kannan 3b7baa1a51 FIX: Don't use topic reply count as answer count in 'Question' schema 2019-02-20 00:35:07 +05:30
Michael Howell 684df58437 Move the QAPage Question into mainEntity
Fixes
https://meta.discourse.org/t/support-schema-org-qapage-and-google-rich-results/33813/14?u=notriddle
2018-12-04 20:04:54 +00:00
Penar Musaraj 7745adb6eb Add FA5 icon support 2018-11-07 21:08:46 -05:00
Sam 7673ed2480 FIX: in some cases first_post could be missing 2018-11-05 10:21:10 +11:00
Sam 54dc2b06ca clean up previous commit that adds answer metadata 2018-11-02 10:15:05 +11:00
Michael Howell 4e00693c76 Add schema.org QAPage metadata (#69)
* Add schema.org QAPage metadata

This is an SEO trick, intended to make results from Discourse forms more
"clickable" by promising a solution at the other end of the link.

https://meta.discourse.org/t/suggestion-add-checkmark-for-solved-threads-to-the-topic-title-as-well/33813

* Use final before-head-close-crawler that got merged

* Fix Sam's problems

* Fix linter errors
2018-11-02 09:38:16 +11:00