Commit Graph

85 Commits

Author SHA1 Message Date
Awais Ansari
bd4ecfdb5b test: fix discussions home fail test cases 2022-02-04 01:51:02 +05:00
Awais Ansari
880c061e9f feat: integrate feeder package 2022-02-03 23:08:24 +05:00
Ahtisham Shahid
0a66e60f37 chore: regenrated package-lock to fix gocd pipeline (#52) 2022-01-27 23:49:16 +05:00
Ahtisham Shahid
a82427d0bc chore: updated frontend-build package to fix gocd build (#51)
* chore: updated frontend-build package to fix gocd build
2022-01-27 17:54:54 +05:00
Kshitij Sobti
9ba94966f1 feat: delete confirmation dialogs [BD-38] [TNL-9351] (#49)
Adds delete confirmation dialogs for posts, comments and replies.
2022-01-27 07:18:43 +00:00
Kaustav Banerjee
f210379a39 fix: selecting different post does not clear comments list (#45) 2022-01-11 05:52:37 +00:00
Kshitij Sobti
55c37db023 fix: remove discussions from the base path [BD-38] [TNL-9347] (#47)
* fix: remove discussions from the base path

Remove the discussions prefix from the base path to avoid potential repeat of that
word the path when deployed.
2022-01-07 04:54:36 +00:00
Felipe Trzaskowski
e4b34c2150 fix: Sort and deduplicate comments when loading the next page [BD-38] [TNL-8657] [BB-5007] (#44)
fix: Sort and deduplicate comments when loading the next page

This fixes the issue with loading the next page of older comments after
posting a new one, which only appended comments to the end, leaving the
newer one behind.

Deduplicating also fixes the related issue, when the new comment would
repeat after paginating to its correct position.
2021-12-30 12:08:07 +00:00
Kshitij Sobti
df455b992e Merge pull request #41 from edx/kshitij/fix-redirects
fix: fixes issues with redirects [BD-38] [TNL-9387]
2021-12-28 11:12:43 +00:00
Kshitij Sobti
2ed5aee9e7 fix: fixes issues with redirects
Currently redirecting doesn't preserve query parameters. This change fixes this and fixes all usages of history.push
2021-12-28 16:37:15 +05:30
Kshitij Sobti
ea1031e2bf Merge pull request #43 from edx/kshitij/fix-topic-search
fix: starting a search redirects you to the all posts page [BD-38] [TNL-9336]
2021-12-28 11:04:05 +00:00
Kshitij Sobti
f31d61b55e Merge pull request #42 from edx/kshitij/fix-thread-counts
fix: topic counts were not included in normalised data [BD-38] [TNL-9403]
2021-12-28 11:03:34 +00:00
Kshitij Sobti
9c7be068ac fix: topic counts were not included in normalised data. 2021-12-10 00:33:05 +05:30
Kshitij Sobti
a6f7187893 fix: starting a search redirects you to the all posts page 2021-12-10 00:23:10 +05:30
Kshitij Sobti
a318bfe517 Merge pull request #38 from edx/kshitij/fix-autosave
fix: fix autosaving  [BD-38] [TNL-9303]
2021-12-09 06:26:33 +00:00
Kshitij Sobti
4763702bd3 fix: fix autosaving 2021-12-09 11:27:41 +05:30
Kshitij Sobti
0646d717ea Merge pull request #39 from edx/kshitij/cohort-support
feat: Fully implements support for cohorts [BD-38] [TNL-9304]
2021-12-08 13:00:37 +00:00
Kshitij Sobti
b07b4043b0 test: tests 2021-12-06 15:45:14 +05:30
Kshitij Sobti
ceea220d62 feat: Fully implements support for cohorts
Uses the discussions settings API to fetch course division configuration and makes the cohort selection dynamic based on if a topic is divided.
2021-12-06 11:15:01 +05:30
Kshitij Sobti
acc466c993 Merge pull request #37 from edx/kshitij/mobile
feat: Make UI more responsive to smaller screens and embeds [BD-38] [TNL-8430] [BB-4420]
2021-12-02 12:24:13 +00:00
Kshitij Sobti
e7a0eba5ef feat: Make UI more responsive to smaller screens and embeds
Updates the UX so that on a smaller screen eitherthe topic/post sidebar is displayed or the post view to give each more space.

Also adds support for an in-context view where the header and navigation is replaced with a sidebar-specific header.
2021-11-25 12:38:11 +05:30
Kshitij Sobti
54131fa13f Merge pull request #36 from edx/kshitij/tnl-8519/breadcrumbs
feat: add new breadcrumb bar for navigation course topics [BD-38] [TNL-8519]
2021-11-24 15:03:58 +00:00
Kshitij Sobti
c9cb8dd0e7 feat: Add new breadcrumb navigation
Adds the new breadcrumb dropdown-based navigation UI. This allows browsing the category and topic structure using dropdowns.
2021-11-23 14:43:53 +05:30
Kshitij Sobti
95bb0de4c0 Merge pull request #32 from edx/kshitij/topic-link
feat: Add topic relation to course structure [BD-38] [TNL-8790]
2021-11-17 03:44:51 +00:00
Kshitij Sobti
7f26f3a6ca feat: Add topic link to discussion post
Refactors the topics code and adds the topic relation box to posts.
2021-11-17 09:11:10 +05:30
Kshitij Sobti
cb09b8c345 Merge pull request #35 from edx/kshitij/anonymous-posting
feat: add support for anonymous posting [BD-38] [TNL-9224]
2021-11-16 08:09:23 +00:00
Kshitij Sobti
18d19369a9 feat: add support for anonymous posting
Hooks up the configuration API to allow posting threads anonymously.
2021-11-16 10:54:48 +05:30
Kshitij Sobti
6bd7ce929d Merge pull request #33 from edx/kshitij/post-ui-updates
feat: Show cohort info in posts [BD-38] [TNL-8694]
2021-11-09 13:58:29 +00:00
Kshitij Sobti
8b188e05b0 Merge pull request #31 from open-craft/maxim/bb-4842-add-tests-topics-view
test: add tests for TopicsView component [BD-38] [TNL-8696] [BB-4842]
2021-11-08 06:21:43 +00:00
Awais Jibran
681bc4de67 Merge pull request #27 from open-craft/felipetrz/bb-4961-tiny-mce-improvements
feat: TinyMCE improvements [BD-38] [TNL-8779] [BB-4961]
2021-11-04 09:47:10 +05:00
Maxim Beder
fc2c528bf7 test: add tests for TopicsView component 2021-11-02 16:38:48 +01:00
Kshitij Sobti
5564ef7549 Merge pull request #30 from open-craft/maxim/bb-4842-add-tests-dicussions-home
test: add simple test for DiscussionsHome [BD-38] [TNL-8696] [BB-4842]
2021-11-02 11:53:02 +00:00
Kshitij Sobti
c1f9b45c6f Merge pull request #26 from open-craft/maxim/bb-4926-add-cohort-selection
feat: add cohort selection [BD-38] [TNL-8713] [BB-4926]
2021-11-02 06:38:33 +00:00
Kshitij Sobti
b03681ce8b Merge pull request #29 from open-craft/maxim/bb-4842-add-tests-actions-dropdown
test: add tests for ActionsDropdown [BD-38] [TNL-8696] [BB-4842]
2021-11-01 10:31:28 +00:00
Felipe Trzaskowski
2138c12eb4 fix: remove the "powered by tiny" text
fix: skin loading with transparent background

For some reason, loading the skin from `skin.min.css` was resulting in
dialogs and menus appearing with a transparent background. Replacing the
import for the non-minified version, `skin.css`, solves the problem.

feat: enable native browser spellchecking

Disabling the TinyMCE context menu is required for the native
spellchecking right click suggestions to work.

feat: enable autosave plugin

Autosaves to local storage every second.
Restores draft when empty.

feat: add new toolbar buttons

Additional toolbar buttons requested: quoting, code formatting, HTML button with text
label, clear formatting, undo/redo

feat: add image button to the toolbar, and load the imagetools plugin

The imagetools only work on images loaded from the same domain.

feat: add image upload capability

Depends on an upload endpoint in the REST API.

feat: enable a11y_advanced_options

This will only affect the image plugin.
For full a11y checking functionality, a premium TinyMCE key is required.

fix: TinyMCE content styles not being properly loaded

Content CSS need to be loaded as raw text, to be included in the TinyMCE
initialization. For this, we need webpack's `raw-loader` as a dependency.

fix: refactor the upload API call into posts/data/api

feat: pass courseId and threadId to the upload API endpoint

fix: rename customized buttons with openedx prefix

fix: use exact version for dependencies

fix: improve jsdoc and update the upload endpoint

fix: simplify TinyMCE side of the upload call

fix: update the upload url
2021-10-29 17:15:27 -03:00
Maxim Beder
35c9dd543c test: add tests for ActionsDropdown 2021-10-29 21:29:20 +02:00
Kshitij Sobti
2f7771e9fc feat: add cohort information to post footer 2021-10-29 19:13:26 +05:30
Maxim Beder
5fafa44b56 feat: add cohort selection
Allow admins select a cohort when creating a new post.

If a user is not a discussions administrator, or if trying to edit
existing post, can't select a cohort (field is hidden).

fix: fix adding threads for topics that haven't been preloaded
2021-10-28 21:51:48 +02:00
Kshitij Sobti
b8b91b9225 Merge pull request #28 from edx/kshitij/fixes
feat: implement new filter bar and UX changes [BD-38] [TNL-9168] [TNL-8787]
2021-10-28 11:22:48 +00:00
Kshitij Sobti
c8c3bb925c feat: Improve UX to match new mockups
Uses new API features for preview, deleting, pinning, and closing. Refactors and improves styling to match new mockups.
Adds the new post filter bar element.
2021-10-28 16:43:28 +05:30
Maxim Beder
491f7cd92c test: add simple test for DiscussionsHome 2021-10-26 03:25:04 +02:00
Kshitij Sobti
1c6a9b24dd Merge pull request #25 from open-craft/felipetrz/bb-4993-add-post-pagination
feat: add pagination for posts [BD-38] [TNL-8810] [BB-4993]
2021-10-21 12:49:08 +05:30
Felipe Trzaskowski
3a71253e34 fix: move the loading indicator and scroll threshold inside the posts list 2021-10-19 11:03:51 -03:00
Felipe Trzaskowski
b8e3b86264 fix: posts pagination is loaded by scrolling
Implemented a ScrollThreshold reusable component to detect scrolling
events, which can be used in the place of a Button.

Removed the i18n entry created for the Button, which is no longer
needed.
2021-10-19 11:01:05 -03:00
Felipe Trzaskowski
1d3827ed10 feat: add posts pagination [BD-38] [TNL-8810] [BB-4993] 2021-10-19 11:01:05 -03:00
Kshitij Sobti
12500dd5e4 Merge pull request #23 from open-craft/maxim/BB-4735-add-comments-within-comments-pagination
feat: add pagination for responses to comments [BD-38] [TNL-8657] [BB-4735]
2021-10-19 10:02:57 +05:30
Ned Batchelder
94e2c548d7 build: use the organization commitlint check 2021-10-07 13:54:06 -04:00
Maxim Beder
93ae8bcffe feat: add pagination to comments responses
This change adds pagination to comments responses.

When viewing a post with a comment or comments that has more than one
page of responses, initially users will only see the first page of the
responses under each comment, and can load more comments by clicking
"load more responses" button under already displayed responses for that
comment.
2021-10-06 18:44:17 +02:00
Kshitij Sobti
f99337080a Merge pull request #22 from edx/kshitij/tnl-8723/questions
feat: add support for question type posts [BD-38] [TNL-8723] [BB-4884]
2021-10-06 14:20:10 +00:00
Kshitij Sobti
d237be78b6 feat: add support for loading endorsed and un-endorsed comments separately for question type posts 2021-10-06 18:26:06 +05:30