Learning Qna #2320

nur-alam · 2026-01-19T11:09:39Z

No description provided.

- Create new qna.ts page component with Q&A mutation handlers - Add initializeQnA function to register Q&A component with Alpine.js - Integrate Q&A page initialization into learning area router - Implement createQnA, replyQna, and deleteQnA mutation methods - Add success/error toast notifications for Q&A operations - Update package-lock.json dependencies - Enable Q&A functionality in learning area templates

…d reply handling

b-l-i-n-d · 2026-01-20T04:36:33Z

assets/src/js/frontend/learning-area/pages/qna.ts

+/**
+ * Q&A Page Component
+ * Handles Q&A question creation in learning area
+ */


This comment is unnecessary.

b-l-i-n-d · 2026-01-20T04:37:23Z

assets/src/js/frontend/learning-area/pages/qna.ts

+          window.location.reload();
+        },
+        onError: (error: Error) => {
+          window.TutorCore.toast.error(error.message || __('Failed to save question', 'tutor'));


use convertToErrorMessage

Suggested change

window.TutorCore.toast.error(error.message || __('Failed to save question', 'tutor'));

window.TutorCore.toast.error(convertToErrorMessage(error));

b-l-i-n-d · 2026-01-20T04:37:35Z

assets/src/js/frontend/learning-area/pages/qna.ts

+          window.location.reload();
+        },
+        onError: (error: Error) => {
+          window.TutorCore.toast.error(error.message || __('Failed to save reply', 'tutor'));


b-l-i-n-d · 2026-01-20T04:38:08Z

assets/src/js/frontend/learning-area/pages/qna.ts

+        },
+      });
+
+      // Q&A delete mutation.


Please remove this comment as this is self-explanatory.

b-l-i-n-d · 2026-01-20T04:38:34Z

assets/src/js/frontend/learning-area/pages/qna.ts

+          }
+        },
+        onError: (error: Error) => {
+          window.TutorCore.toast.error(error.message || __('Failed to delete Q&A', 'tutor'));


use convertToErrorMessage

Suggested change

window.TutorCore.toast.error(error.message || __('Failed to delete Q&A', 'tutor'));

window.TutorCore.toast.error(convertToErrorMessage(error));

b-l-i-n-d · 2026-01-20T04:46:26Z

templates/learning-area/subpages/qna-single.php

+					</div>
+					<?php if ( $current_user_id == $answer->user_id || current_user_can( 'manage_tutor' ) ) : ?>
+						<div x-data="tutorPopover({ placement: 'bottom-end', offset: 4 })" class="tutor-ml-auto">
+							<button class="tutor-btn tutor-btn-ghost tutor-btn-icon tutor-btn-x-small" x-ref="trigger" @click="toggle()">


Use php Button component

b-l-i-n-d · 2026-01-20T04:47:37Z

templates/learning-area/subpages/qna-single.php

+						</div>
+					</div>
+					<?php if ( $current_user_id == $answer->user_id || current_user_can( 'manage_tutor' ) ) : ?>
+						<div x-data="tutorPopover({ placement: 'bottom-end', offset: 4 })" class="tutor-ml-auto">


If this can be done with php Popover component, then please use that.

b-l-i-n-d · 2026-01-20T04:48:17Z

templates/learning-area/subpages/qna.php

+			?>
+			<div class="tutor-input-field">
+				<div class="tutor-input-wrapper">
+					<input 


Use php component

b-l-i-n-d · 2026-01-20T04:49:41Z

templates/learning-area/subpages/qna.php

+	<form 
+		class="tutor-discussion-form tutor-p-6 tutor-border-b tutor-qna-form" 
+		x-data="{ ...tutorForm({ id: '<?php echo esc_attr( $tutor_course_id ); ?>' }), focused : false }"
+		@submit.prevent="handleSubmit((data) => createQnaMutation?.mutate({...data, course_id: <?php echo esc_html( $tutor_course_id ); ?> }))($event)"


Pass course_id as a default value.

b-l-i-n-d · 2026-01-20T04:50:11Z

templates/learning-area/subpages/qna.php

+				$content      = strlen( $content ) > 100 ? substr( $content, 0, 100 ) . '...' : $content;
+				?>
+				<div class="tutor-discussion-card" @click="window.location.href = '<?php echo esc_url( $question_url ); ?>'" style="cursor: pointer;">
+					<div class="tutor-avatar tutor-avatar-32">


Use php Avatar component

- Import and use convertToErrorMessage utility for consistent error formatting

harunollyo · 2026-01-20T06:03:14Z

templates/learning-area/subpages/qna-single.php

 * @link https://themeum.com
 * @since 4.0.0
 */



Prevent direct access

defined( 'ABSPATH' ) || exit;

harunollyo · 2026-01-20T06:03:39Z

templates/learning-area/subpages/qna-single.php

+
+// Get course ID and question ID.
+global $tutor_course_id;
+$question_id = Input::get( 'question_id', null, Input::TYPE_INT );


Use default 0

harunollyo · 2026-01-20T06:04:16Z

templates/learning-area/subpages/qna-single.php

+$order_by    = Input::get( 'order', 'DESC' );
+
+// Get question data.
+$question = tutor_utils()->get_qa_question( (int) $question_id );


No need to cast. It's already int by Input::TYPE_INT

harunollyo · 2026-01-20T06:07:41Z

templates/learning-area/subpages/qna-single.php

-					</button>
-					5
+
+	<?php if ( is_array( $answers ) && count( $answers ) ) : ?>


Instead of this is_array( $answers ) && count( $answers ) use

tutor_utils()->count( $answers );

harunollyo · 2026-01-20T06:08:18Z

templates/learning-area/subpages/qna.php

 * @link https://themeum.com
 * @since 4.0.0
 */



Prevent direct access

harunollyo · 2026-01-20T06:15:06Z

templates/learning-area/subpages/qna.php

-			);
+			Sorting::make()
+				->order( Input::get( 'order', 'DESC' ) )
+				->label_asc( __( 'Newest First', 'tutor' ) )


Default value is same as you passed. No need to call label_asc and label_desc

harunollyo · 2026-01-20T06:16:29Z

templates/learning-area/subpages/qna.php

+		<?php
+		Pagination::make()
+			->current( $current_page )
+			->total( (int) $total_items )


No need to cast int

harunollyo · 2026-01-20T06:16:49Z

templates/learning-area/subpages/qna-single.php

+			<?php
+			Sorting::make()
+				->order( Input::get( 'order', 'DESC' ) )
+				->label_asc( __( 'Newest First', 'tutor' ) )


Default value is same as you passed. No need to call label_asc and label_desc

sazedul-haque · 2026-01-20T07:03:52Z

assets/src/scss/frontend/learning-area/layout/_sidebar.scss

-  position: fixed;
-  height: calc(100vh - 62px);
-  top: 62px;
+  position: absolute;


Remove this.

sazedul-haque · 2026-01-20T07:05:00Z

templates/learning-area/index.php

 				if ( $subpage ) {
 					$subpage_template = tutor_get_template( 'learning-area.subpages.' . $subpage );
-					if ( file_exists( $subpage_template ) ) {
+					if ( $question_id ) {


This can be managed inside qna.php template.

- Refactor learning-area index template to handle question_id routing in qna-single subpage - Update Sorting component usage by removing custom labels - Improve answer count validation using tutor_utils()->count() helper - Add SearchFilter component import and implement search functionality in Q&A page - Add ABSPATH check for security in qna-single template - Consolidate question_id routing logic into qna-single subpage for better separation of concerns - Add Avatar component import to qna template

- Add Popover component import to qna-single.php - Replace manual x-data popover implementation with Popover::make() component for question actions menu - Replace manual x-data popover implementation with Popover::make() component for reply actions menu - Update SearchFilter placeholder text from 'Search items...' to 'Search questions, topics...' - Update textarea placeholder text from 'Ask a question...' to 'Asked questions...' - Fix indentation inconsistency in SearchFilter component call - Simplifies popover markup by using component abstraction instead of manual Alpine.js directives

- Add `defined( 'ABSPATH' ) || exit;` security check to both qna.php and qna-single.php templates - Replace manual form with SearchFilter component using UrlHelper for cleaner URL handling - Update form submission to use defaultValues in tutorForm configuration - Remove unnecessary hidden input loop in favor of component-based hidden_inputs parameter

… both question and answer timestamps to use consistent sprintf pattern

nur-alam added 8 commits January 13, 2026 16:47

Merge branch '4.0.0-dev' of github.com:themeum/tutor into 4.0.0-dev

63fa54e

Merge branch '4.0.0-dev' of github.com:themeum/tutor into 4.0.0-dev

147cbab

load template of qna single page

216f9a7

Merge branch '4.0.0-dev' of github.com:themeum/tutor into courses-qna

35fe1e0

Add clickable discussion card navigation in Q&A

bbd87d9

Merge branch '4.0.0-dev' of github.com:themeum/tutor into courses-qna

e25dd27

fix(qna): Remove debug console.log from deleteQnA method

241f326

nur-alam requested review from b-l-i-n-d, harunollyo and sazedul-haque and removed request for harunollyo January 19, 2026 11:09

nur-alam added 5 commits January 19, 2026 21:23

fix(qna): Update URL parameter name in deleteQnA mutation

352a3f2

update(qna): Add context parameter to delete mutation for question an…

7dcb1dc

…d reply handling

fix(qna): Rename mutation callback parameter from variables to payload

3ea3e4a

Merge branch '4.0.0-dev' of github.com:themeum/tutor into courses-qna

71560a9

fix(learning-area): Rename QnA function to follow camelCase convention

b8f4930

b-l-i-n-d requested changes Jan 20, 2026

View reviewed changes

harunollyo added the 4.0.0 Tutor v4.w0w label Jan 20, 2026

refactor(qna): Standardize error handling and update button component

603b49d

- Import and use convertToErrorMessage utility for consistent error formatting

harunollyo requested changes Jan 20, 2026

View reviewed changes

sazedul-haque requested changes Jan 20, 2026

View reviewed changes

nur-alam added 7 commits January 20, 2026 13:45

Merge branch '4.0.0-dev' of github.com:themeum/tutor into courses-qna

c2ba9e7

style(discussion-card): Remove unused search wrapper styles

d4ae050

fix(qna): Standardize question_id input handling with default value

d689c4d

refactor(qna): Replace inline avatar markup with Avatar component

83bacac

nur-alam added 4 commits January 20, 2026 16:46

Merge branch '4.0.0-dev' of github.com:themeum/tutor into courses-qna

388b85d

refactor(qna): Replace inline markup with Button component and Update…

1dd634a

… both question and answer timestamps to use consistent sprintf pattern

refactor(qna): Add frontend context parameter to answer retrieval

95ac3a7

Merge branch '4.0.0-dev' of github.com:themeum/tutor into courses-qna

cd41e74

	window.TutorCore.toast.error(error.message \|\| __('Failed to save question', 'tutor'));
	window.TutorCore.toast.error(convertToErrorMessage(error));

Learning Qna #2320

Are you sure you want to change the base?

Learning Qna #2320

Conversation

nur-alam commented Jan 19, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants