Skip to content

Conversation

@hassan-raza-1
Copy link
Contributor

@hassan-raza-1 hassan-raza-1 commented Mar 25, 2025

Description

We need to implement group notifications for staff regarding ORA submissions.

Supporting information

Ticket: https://2u-internal.atlassian.net/browse/INF-1838

Testing instructions

Submit an Open Response for staff review from two different users on the same XBlock. Verify that the staff user receives a single notification for both submissions.

@hassan-raza-1 hassan-raza-1 force-pushed the hraza/group_ora_notifications branch from 7bb48ac to d79d676 Compare March 25, 2025 10:02
@hassan-raza-1 hassan-raza-1 changed the title ADD: Group ora staff notifications [DRAFT] feat: Group ora staff notifications [DRAFT] Mar 25, 2025
@hassan-raza-1 hassan-raza-1 force-pushed the hraza/group_ora_notifications branch 2 times, most recently from e787bf8 to de6e7e0 Compare March 25, 2025 11:45
@hassan-raza-1 hassan-raza-1 changed the title feat: Group ora staff notifications [DRAFT] feat: Group ora staff notifications Mar 26, 2025
@hassan-raza-1 hassan-raza-1 force-pushed the hraza/group_ora_notifications branch from de6e7e0 to 07b6110 Compare March 26, 2025 09:21
@hassan-raza-1 hassan-raza-1 force-pushed the hraza/group_ora_notifications branch from 07b6110 to f985ff7 Compare March 26, 2025 09:46
'is_core': False,
'info': '',
'web': False,
'web': True,
Copy link
Contributor

Choose a reason for hiding this comment

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

Changing defaults would not work on existing preferences , we should do this on a separate PR .
It would require a version change and a name change for this preference type.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

sure, Let me revert this change

@hassan-raza-1 hassan-raza-1 merged commit 40c82c2 into master Apr 9, 2025
49 checks passed
@hassan-raza-1 hassan-raza-1 deleted the hraza/group_ora_notifications branch April 9, 2025 09:34
@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production.

@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

1 similar comment
@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

irtazaakram pushed a commit that referenced this pull request Apr 10, 2025
* feat: Group ora staff notifications

* fix: Revert default change for ora staff notifications

* fix: update edx-ora2 version
farhan pushed a commit that referenced this pull request Apr 17, 2025
* feat: Group ora staff notifications

* fix: Revert default change for ora staff notifications

* fix: update edx-ora2 version
tonybusa pushed a commit to tonybusa/edx-platform that referenced this pull request Apr 23, 2025
* feat: Group ora staff notifications

* fix: Revert default change for ora staff notifications

* fix: update edx-ora2 version
UsamaSadiq pushed a commit that referenced this pull request May 14, 2025
* feat: Group ora staff notifications

* fix: Revert default change for ora staff notifications

* fix: update edx-ora2 version
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants