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.
* build: remove unused @edx/reactifex package
Remove @edx/reactifex package from devDependencies as it is no longer
needed. Translation extraction functionality has been verified to work
correctly without this dependency.
Co-Authored-By: Claude <noreply@anthropic.com>
* build: Drop the push_translations target.
That work is now done by the openedx-translations repo.
---------
Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: Adolfo R. Brandes <adolfo@axim.org>
Earlier versions of @openedx/frontend-build used on older version of
'sharp', which caused intermittent installation issues. The version of
'sharp' was updated in @openedx/frontend-build to fix these issues, so
the frontend-build version can be updated here, to fix the issues in
this project too. See
https://github.com/openedx/frontend-build/issues/664 and
https://github.com/openedx/frontend-build/pull/665 for more information.
The frontend-build dependency was updated by:
```
npm install --package-lock-only @openedx/frontend-build
```
Private-ref: https://tasks.opencraft.com/browse/BB-9953
* 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.
