Skip to content

Conversation

@efortish
Copy link
Contributor

@efortish efortish commented Sep 26, 2025

Related issue: https://github.com/fccn/nau-technical/issues/661

Sumary

This PR introduces cherry-picks needed by edx-platform to run custom functionalities of NAU edx-platform.

Notes

This commit 7e14e52 has an upstream PR in progress, the commit was cherry-picked but once the upstream PR is merged we will have 2 options:

  • Use the cherry-pick (it currently works with the NAU configuration)
  • Backport the upstream solution, this requires taking a look due the solution in upstream changes the behavior of setup the custom fields without using the setting. (the setting was not accepted by the community)

These three commits are similar. Teak uses a default django-require requirement, so the commits are no longer needed.


There are a few different commits that were not cherry-picked because Teak already provides their functionality.

b5fa855, 027445f, 0e3f12a, 464f36a, cf9559f, db3900d, 85706cc, bd5bf83, 2e942d5, 08c45cb, fc13128, 1870293, 88bcafe, 5f3089c, 3607bd6, 83a164a, 1d1424e, 9a19347, e85fee6, 7611160, 3c69457


Commits cherrypicked:

d735bed , 6a1f6a9, 13a7bbf, 16b4360, c22534a, 3b21ebf, 51bc82f, f6e74d4, ffbcd43, 1c6634e, 3964db5, 82f68df, ea17d5c, 8822fdb, 235d77f, df562d3, b7107c1, bdae625, 1c503d7, 821cc55, e0c4e09, 6c85155, 9114106, a53207e, 5063021, baf522a

One of these commits was cherry-pick --skip, due duplicate logic, that's why the PR has 25 commits but 26 entrys.


MaferMazu and others added 25 commits September 26, 2025 11:54
The NAU custom field data_authorization should be put
after the optional fields.
The motivation is that we need to add multiple authorization
checkboxes to the register page, requested by the DPO.

fccn/nau-technical#83
Get Redwood RegisterView.js file and apply custom NAU code.

fccn/nau-technical#337
Just copy the missing code that was lost during upgrade NAU code for redwood.
Add the user certificate to the CertificateRenderStarted Open edX filter context, so it is possible to have complex filtering and applying
custom logic to the course certificate presentation.

fccn/nau-technical#667
Add `enrollment_date` column on the csv file of all students enrolled
in a course.

Allow site operators to include on the export of profile information as CSV
custom fields if the platform has an extending User model.
This can be used if you have an extended model that include for example
an university student number and site operator want to export the
student number on the student profile information CSV.

GN-914
@efortish efortish marked this pull request as draft September 26, 2025 17:45
@efortish efortish requested a review from igobranco October 7, 2025 17:29
@efortish efortish marked this pull request as ready for review October 7, 2025 17:33
@igobranco igobranco requested a review from sandroscosta October 8, 2025 08:12
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.

10 participants