Wrap learnerPostsLink creation in useMemo with guard to prevent
'Missing :learnerUsername param' error. The generatePath function
was being called unconditionally during render even when the link
wouldn't be displayed, causing errors when author was null, undefined,
or the 'anonymous' string.
The fix ensures generatePath is only called when showUserNameAsLink
is true, which validates that author is a valid username.
This reverts commit 909d133acc.
See https://github.com/openedx/edx-platform/issues/37402 for details
We leave the original PR's changes to Confirmation.jsx in place
because they are OK and are now tangled in with recent changes
and more difficult to revert.
* feat: updated v2 captcha to v3 in post editor
* feat: added google captcha v3 for comment
* test: added test cases
* test: added test case to update the post
* test: updated test case for preview node
* test: updated test case for comment error
* test: removed mock file
* fix: removed comments
---------
Co-authored-by: sundasnoreen12 <sundasnoreen12@gmail.com>
* feat: added rate limit dialogue
* test: added test cases for post comment
* test: added test for content creation dialogue
* test: added test cases for empty topics
* test: added test cases
* feat: added content rate limit dialogue on post API for post and comment
* fix: fixed editor close issue on submit
* test: addd test cases
* feat: Modified discussions FE so that ONLY users with verified emails can create content
* feat: added comment and response functionality
* test: fixed test cases
* refactor: refactor code and added HOC
* test: added test cases
* test: added test cases for failed and denied
* feat: added states for button
* refactor: added callback function
* test: added test case to close the dialogue
* refactor: refactor function
* feat: added captcha to discussion post creation
* feat: added captcha for comment and response
* fix: removed learner check
* test: fixed test cases
* fix: removed comment and added check for empty sitekey
* fix: fixed translation issue
* test: added test cases for recaptcha
* test: should allow posting a comment with CAPTCHA
* test: added submit post test cases
* test: test edge cases for api
* test: added test cases for react-google-recaptcha
* test: added test case for default values for captcha
* fix: removed unused catch
---------
Co-authored-by: sundasnoreen12 <sundasnoreen12@gmail.com>
Co-authored-by: Awais Ansari <awais.ansari63@gmail.com>
Adds defensive null checks when accessing post properties in the posts
forEach loop to prevent potential errors in the MFE discussion sidebar.
This addresses the issue reported in #751.
"Add a post" button was visible even though the banner says that posting is
restricted. This change helps in removing the button when posting is restricted.
Signed-off-by: Farhaan Bukhsh <farhaan@opencraft.com>
