Vue/padv 1558 #35999

Nekenhei · 2024-12-09T21:03:10Z

No description provided.

We follow the instructions from: https://openedx.atlassian.net/wiki/spaces/COMM/pages/19662426/Process+to+Create+an+Open+edX+Release#Make-release-specific-changes-on-the-release-branch

build: kickoff the Olive release

This workflow wasn't updated when we updated shards for self-hosted runners, so updating in this commit

…ts-gh-hosted build: run tests for open-release branches on GH hosted runners

…3-xss-openredirect-fix-olive fix: XSS and open redirect vulnerability

* fix: studio submit handler * style: remove extra line Co-authored-by: Matjaz Gregoric <[email protected]>

BREAKING CHANGE: disallowed HTML tags (e.g. <script>) will no longer be rendered in LMS and Studio.

…rag_and_drop_olive feat!: update Drag and Drop v2 XBlock to prevent XSS vulnerabilities (Olive backport)

…6-refreshable-jwt-token fix: Fix retiring user auth models on disable event

…live fix: Fix retiring user auth models on disable event

This change documents some of the workings of the discussions app for reference.

Course topics are now created by traversing the entire course structure from top to bottom instead of starting at the sequential level and then moving up or down as needed. This also introduces a lot of debug logs to pontetially find the reason why under some circumstances new units don't get processed and end up without a discussions topic.

) When running in a sharded MongoDB setup it's possible that querying the modulestore right after the course publish signal will not return the latest data. This commit adds a delay similar to the one used in other places in the codebase for a similar reason.

Co-authored-by: adeel.tajamul <[email protected]>

…penedx#31048)

* fix: updated topic sorting for discussions MFE * feat: updated topic sorting test case

…uctor paced courses

…fication

Co-authored-by: adeel.tajamul <[email protected]>

…1252) Co-authored-by: adeel.tajamul <[email protected]>

…1275)

…dres

PADV-1511 - Add setting to store SAML IDP id values.

PADV-1556 Create custom form for registration extra fields.

…frame-heigth fix: courseware-chromeless.html iframe height

PADV-1418 feat: adds a new custom content jsonfield to ccx model

PADV 1580 - Create annotatorjs plugin for AI select and summarize

PADV 1584 - Add summarize functionality to plugin

PADV 1684 - Add feature flag for LLM summarize for frontend

…ent devices.

PADV-1593: Expand Annotation Textbox

PADV-1593 fix: revert modal resize removal

PADV 1853 - Fix LLM summarize feature flag not working

openedx-webhooks · 2024-12-09T21:03:16Z

Thanks for the pull request, @Nekenhei!

What's next?

Please work through the following steps to get your changes ready for engineering review:

🔘 Get product approval

If you haven't already, check this list to see if your contribution needs to go through the product review process.

If it does, you'll need to submit a product proposal for your contribution, and have it reviewed by the Product Working Group.
- This process (including the steps you'll need to take) is documented here.
If it doesn't, simply proceed with the next step.

🔘 Provide context

To help your reviewers and other members of the community understand the purpose and larger context of your changes, feel free to add as much of the following information to the PR description as you can:

Dependencies

This PR must be merged before / after / at the same time as ...
Blockers

This PR is waiting for OEP-1234 to be accepted.
Timeline information

This PR must be merged by XX date because ...
Partner information

This is for a course on edx.org.
Supporting documentation
Relevant Open edX discussion forum threads

🔘 Get a green build

If one or more checks are failing, continue working on your changes until this is no longer the case and your build turns green.

🔘 Let us know that your PR is ready for review:

Who will review my changes?

This repository is currently maintained by @openedx/wg-maintenance-edx-platform. Tag them in a comment and let them know that your changes are ready for review.

Where can I find more information?

If you'd like to get more details on all aspects of the review process for open source pull requests (OSPRs), check out the following resources:

When can I expect my changes to be merged?

Our goal is to get community contributions seen and reviewed as efficiently as possible.

However, the amount of time that it takes to review and merge a PR can vary significantly based on factors such as:

The size and impact of the changes that it introduces
The need for product review
Maintenance status of the parent repository

💡 As a result it may take up to several weeks or months to complete a review and merge your PR.

mtyaka and others added 30 commits October 11, 2022 19:54

build: kickoff the Olive release

73d7560

We follow the instructions from: https://openedx.atlassian.net/wiki/spaces/COMM/pages/19662426/Process+to+Create+an+Open+edX+Release#Make-release-specific-changes-on-the-release-branch

Merge pull request openedx#31144 from openedx/mtyaka/olive-kickoff

eed0dca

build: kickoff the Olive release

build: run tests for open-release branches on GH hosted runners

b2305b4

build: update shard in unit test worflow for forks

df4f843

This workflow wasn't updated when we updated shards for self-hosted runners, so updating in this commit

Merge pull request openedx#31310 from openedx/iamsobanjaved/olive-tes…

9bdba25

…ts-gh-hosted build: run tests for open-release branches on GH hosted runners

fix: XSS and open redirect vulnerability

39b7b2b

Merge pull request openedx#31305 from openedx/sajjad/backport-van-113…

de4d748

…3-xss-openredirect-fix-olive fix: XSS and open redirect vulnerability

fix: studio submit handler (olive backport) (openedx#31219)

203bf9f

* fix: studio submit handler * style: remove extra line Co-authored-by: Matjaz Gregoric <[email protected]>

feat!: update Drag and Drop v2 XBlock to prevent XSS vulnerabilities

903ea00

BREAKING CHANGE: disallowed HTML tags (e.g. <script>) will no longer be rendered in LMS and Studio.

Merge pull request openedx#31353 from open-craft/agrendalath/update_d…

3dbb6c1

…rag_and_drop_olive feat!: update Drag and Drop v2 XBlock to prevent XSS vulnerabilities (Olive backport)

fix: Fix retiring user auth models on disable event

a080367

Merge pull request openedx#31401 from openedx/aakbar/backport-SEC-146…

e6636f0

…6-refreshable-jwt-token fix: Fix retiring user auth models on disable event

fix: Fix retiring user auth models on disable event

220bc72

Merge pull request openedx#31409 from open-craft/keith/backport-jwt-o…

05e6382

…live fix: Fix retiring user auth models on disable event

docs: Document the discussions app (openedx#31045)

7b4975c

This change documents some of the workings of the discussions app for reference.

fix: removed switch experience bar from legacy (openedx#31138)

c459d6c

Co-authored-by: adeel.tajamul <[email protected]>

feat: add "count_flagged" query param to the learner thread endpoint (o…

470b5d7

…penedx#31048)

fix: updated topic sorting for discussions MFE (openedx#31124)

10b5849

* fix: updated topic sorting for discussions MFE * feat: updated topic sorting test case

fix: filter out discussion topics that have not started yet for instr…

00da61d

…uctor paced courses

refactor: remove todo to add tracking for reported content email noti…

fdedd9b

…fication

feat: add discussion reported and unreported events

ef6bda1

feat: added post filters in learners tab api (openedx#31191)

259ff32

Co-authored-by: adeel.tajamul <[email protected]>

feat: added sort by recent activity for activity stats api (openedx#3…

af20df0

…1252) Co-authored-by: adeel.tajamul <[email protected]>

feat: added not responded filter for fetch thread list api (openedx#3…

7ece651

…1275)

fix: legacy coverage and add missing properties for reported events

173ae15

fix: add fix for search event for learner search in discussions

dd68ade

feat: add discussion event coverage for legacy experience.

1314cdb

fix: thread will only be returned for matching course ID (openedx#31340)

afe259a

anfbermudezme and others added 23 commits July 16, 2024 00:28

chore: update py2neo version.

1710fc1

Merge pull request #126 from Pearson-Advance/pearson-release/olive-an…

cde1ca8

…dres

feat: improve the way we handle the IES IDP id values.

975f8f9

Merge pull request #127 from Pearson-Advance/pearson/PADV-1511

1ac574d

PADV-1511 - Add setting to store SAML IDP id values.

feat: registration extension form site awareness

66471bb

Merge pull request #129 from Pearson-Advance/vue/PADV-1556

4ae1637

PADV-1556 Create custom form for registration extra fields.

fix: courseware-chromeless.html iframe height

44d1c52

Merge pull request #132 from Pearson-Advance/kuipumu/fix-courseware-i…

be6e128

…frame-heigth fix: courseware-chromeless.html iframe height

feat: adds a new custom content jsonfield to ccx model

ba49378

Merge pull request #134 from Pearson-Advance/vue/PADV-1418-fix

d780110

PADV-1418 feat: adds a new custom content jsonfield to ccx model

feat: adding summarize plugin for annotatorjs

2d350f5

Merge pull request #135 from Pearson-Advance/vue/PADV-1580.re-opened

f080f01

PADV 1580 - Create annotatorjs plugin for AI select and summarize

feat: adding backend connection to summarize annotatorjs plugin

c544394

Merge pull request #137 from Pearson-Advance/vue/PADV-1584.re-opened

6debc75

PADV 1584 - Add summarize functionality to plugin

feat: add feature flag for LLM summarize

ba1191f

Merge pull request #136 from Pearson-Advance/vue/PADV-1684.re-opened

cdba011

PADV 1684 - Add feature flag for LLM summarize for frontend

style: expand annotation textarea and apply responsiveness for differ…

31f648d

…ent devices.

Merge pull request #138 from Pearson-Advance/vue/PADV-1593

08bf1d3

PADV-1593: Expand Annotation Textbox

fix: revert modal resize removal

4be0148

Merge pull request #139 from Pearson-Advance/vue/PADV-1593.fix

7a4e5b0

PADV-1593 fix: revert modal resize removal

fix: removing llm_summarize from default annotatorjs plugins

8348928

Merge pull request #140 from Pearson-Advance/vue/PADV-1853

4530cd0

PADV 1853 - Fix LLM summarize feature flag not working

chore: update edx-sga version

5697c6c

Nekenhei requested review from a team as code owners December 9, 2024 21:03

openedx-webhooks added the open-source-contribution PR author is not from Axim or 2U label Dec 9, 2024

Nekenhei closed this Dec 9, 2024

Nekenhei deleted the vue/PADV-1558 branch December 9, 2024 21:04

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Vue/padv 1558 #35999

Vue/padv 1558 #35999

Nekenhei commented Dec 9, 2024 •

edited

Loading

openedx-webhooks commented Dec 9, 2024

Vue/padv 1558 #35999

Vue/padv 1558 #35999

Conversation

Nekenhei commented Dec 9, 2024 • edited Loading

openedx-webhooks commented Dec 9, 2024

What's next?

🔘 Get product approval

🔘 Provide context

🔘 Get a green build

🔘 Let us know that your PR is ready for review:

Who will review my changes?

Where can I find more information?

When can I expect my changes to be merged?

Nekenhei commented Dec 9, 2024 •

edited

Loading