Skip to content

feat: add Feedback component#66

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

feat: add Feedback component#66
andrey-canon merged 1 commit intoopen-release/teak.nelpfrom
teak-mig/FUTUREX-1282

Conversation

@andrey-canon
Copy link
Collaborator

@andrey-canon andrey-canon commented Oct 1, 2025

Description

Add fronten-essentials Feedback component. This is a migration pr of #38

Issue # 1282

How to test

  1. Complete a course, If the course is a grading course you have to get the passing grade otherwise you have to complete the 100% of the course, if the course has been set with any assignment type the course will be consider as a grading course
  2. Go to the home course page
  3. Complete the feedback form
  4. Click submit
  5. Check database feedback record /admin/eox_nelp/feedbackcourse

@andrey-canon andrey-canon force-pushed the teak-mig/FUTUREX-1282 branch from a6b5142 to fa20cf0 Compare October 1, 2025 21:48
@andrey-canon andrey-canon changed the base branch from teak-mig/FUTUREX-1437 to teak-mig/FUTUREX-1284 October 1, 2025 21:49
@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-1282 branch from fa20cf0 to 626aa83 Compare October 1, 2025 22:41
@andrey-canon andrey-canon requested a review from johanseto October 1, 2025 23:00
@andrey-canon andrey-canon force-pushed the teak-mig/FUTUREX-1282 branch from 626aa83 to 946b1b8 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.

I have a question. I guess Translations comes in other PRs?

The functionality its ok in my proto teak env.

Screencast.from.07-10-25.16.08.25.webm

@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-1282 branch from 946b1b8 to f824613 Compare October 9, 2025 15:52
@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 force-pushed the teak-mig/FUTUREX-1282 branch from f824613 to d50df3c Compare October 9, 2025 16:36
@andrey-canon andrey-canon changed the base branch from teak-mig/FUTUREX-1284 to open-release/teak.nelp October 9, 2025 16:38
@andrey-canon andrey-canon merged commit 2c2b16f 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:41 — with GitHub Actions Inactive
@andrey-canon andrey-canon temporarily deployed to open-release/teak.nelp October 9, 2025 16:41 — 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