Skip to content

feat: add like-dislike component#63

Merged
andrey-canon merged 1 commit intoopen-release/teak.nelpfrom
teak-mig/FUTUREX-1284
Oct 9, 2025
Merged

feat: add like-dislike component#63
andrey-canon merged 1 commit intoopen-release/teak.nelpfrom
teak-mig/FUTUREX-1284

Conversation

@andrey-canon
Copy link
Collaborator

Description

Adds the like-dislike component in the Sequence file. Migration pr of #35
Issue # 1284

How to test

  1. Add to your platform settings MFE_CONFIG["COURSE_EXPERIENCE_API_URL"] = "http://local.openedx.io:8000/eox-nelp/api/experience/v1"
  2. Go to a unit
  3. press like or dislike
  4. Reload and verify previous state, alternative you can check the admin panel /admin/eox_nelp/likedislikeunit

Expected result

image

@andrey-canon andrey-canon requested a review from johanseto October 1, 2025 18:15
@andrey-canon andrey-canon force-pushed the teak-mig/FUTUREX-1284 branch 2 times, most recently from a6770a6 to 57e3cf5 Compare October 1, 2025 18:36
@andrey-canon andrey-canon force-pushed the teak-mig/FUTUREX-1437 branch from e1d4824 to e4b37f5 Compare October 1, 2025 22:38
@andrey-canon andrey-canon force-pushed the teak-mig/FUTUREX-1284 branch from 57e3cf5 to 268bfcc Compare October 1, 2025 22:39
@andrey-canon andrey-canon force-pushed the teak-mig/FUTUREX-1437 branch from e4b37f5 to 4f35177 Compare October 7, 2025 16:57
@andrey-canon andrey-canon force-pushed the teak-mig/FUTUREX-1284 branch from 268bfcc to 1e68f4a Compare October 7, 2025 16:57
Copy link

@johanseto johanseto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Working in my teak env

Screencast.from.07-10-25.16.00.43.webm

{
"path": "dist/*.js",
"maxSize": "1450kB"
"maxSize": "1536kB"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change is very random xD

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

witchcraft hahah, basically we are adding a new component that means that the result will be heavier that why we have to increase the limit, the number is just the conversion that means 1.5MB

@andrey-canon andrey-canon force-pushed the teak-mig/FUTUREX-1437 branch from 4f35177 to 58d8957 Compare October 9, 2025 15:51
@andrey-canon andrey-canon force-pushed the teak-mig/FUTUREX-1284 branch from 1e68f4a to c64646a Compare October 9, 2025 15:51
@andrey-canon andrey-canon force-pushed the teak-mig/FUTUREX-1437 branch from 58d8957 to 134f9a0 Compare October 9, 2025 16:08
@andrey-canon andrey-canon force-pushed the teak-mig/FUTUREX-1284 branch from c64646a to 896234e Compare October 9, 2025 16:27
@andrey-canon andrey-canon changed the base branch from teak-mig/FUTUREX-1437 to open-release/teak.nelp October 9, 2025 16:28
@andrey-canon andrey-canon merged commit 44bcd7c into open-release/teak.nelp Oct 9, 2025
4 checks passed
@andrey-canon andrey-canon temporarily deployed to open-release/teak.nelp October 9, 2025 16:35 — with GitHub Actions Inactive
@andrey-canon andrey-canon temporarily deployed to open-release/teak.nelp October 9, 2025 16:35 — with GitHub Actions Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants