credentials

credentials copied to clipboard

build: Adding django linter for migrations.

[{"_id":"636f69e9e56470537285984e","body":"Exciting work! Let me know when it's ready for review.","issue_id":1667671331950,"origin_id":1242488015,"user_origin_id":1400344,"create_time":1662758286,"update_time":1662758286,"id":1668245993949,"updated_at":"2022-11-12T09:39:53.948000Z","created_at":"2022-11-12T09:39:53.948000Z"},{"_id":"636f69e9e56470537285984f","body":"Hi @awais786 and @UsamaSadiq - Just checking in- is this ready for review?","issue_id":1667671331950,"origin_id":1291388156,"user_origin_id":100873803,"create_time":1666749408,"update_time":1666749408,"id":1668245993953,"updated_at":"2022-11-12T09:39:53.952000Z","created_at":"2022-11-12T09:39:53.952000Z"},{"_id":"636f69e9e564705372859850","body":"Hi @mphilbrick211 \nThis PR has been created as a proof of concept for the migration-linter tool.\nwe are waiting for the final decision on the approach by the ArchBOM team. Once the approach is finalised, we'll update the status of the PR and will merge it. ","issue_id":1667671331950,"origin_id":1291389359,"user_origin_id":41958659,"create_time":1666749554,"update_time":1666749554,"id":1668245993956,"updated_at":"2022-11-12T09:39:53.955000Z","created_at":"2022-11-12T09:39:53.955000Z"},{"_id":"66a4e894a6309140e005d446","body":"Hi @awais786 and @UsamaSadiq -- this PR hasn't been updated in awhile and is still a draft. \r\n\r\nI am planning on closing this tomorrow. Please let me know if you're planning on picking this up again soon and we need to continue to keep it open.","issue_id":1667671331950,"origin_id":1332693305,"user_origin_id":3229735,"create_time":1669839610,"update_time":1669839610,"id":1722083476970,"updated_at":"2024-07-27T12:31:16.969000Z","created_at":"2024-07-27T12:31:16.969000Z"},{"_id":"66a4e894a6309140e005d447","body":"Hi @justinhynes \r\nThis PR has been created an an example PR for a POC of a new tool `django-migration-linter`. \r\nI'd like to request keeping these PRs open for a while so we can do the final discussion and decide the further approach on it. ","issue_id":1667671331950,"origin_id":1334835461,"user_origin_id":41958659,"create_time":1669964437,"update_time":1669964437,"id":1722083476974,"updated_at":"2024-07-27T12:31:16.973000Z","created_at":"2024-07-27T12:31:16.973000Z"},{"_id":"66a4e894a6309140e005d448","body":"@UsamaSadiq Is there a ticket for this that we can watch? I would like to see us try something on this front.","issue_id":1667671331950,"origin_id":1335941601,"user_origin_id":1400344,"create_time":1670021952,"update_time":1670021952,"id":1722083476977,"updated_at":"2024-07-27T12:31:16.977000Z","created_at":"2024-07-27T12:31:16.977000Z"},{"_id":"66a4e894a6309140e005d449","body":"Hi @hurtstotouchfire \r\nWe don't have any public issue created for this POC yet. A 2u-internal documents for the POC findings have been created for review right now. \r\nI'll update the status on the PR as soon as we get a green signal or any comments on this approach. ","issue_id":1667671331950,"origin_id":1339233189,"user_origin_id":41958659,"create_time":1670328874,"update_time":1670328874,"id":1722083476980,"updated_at":"2024-07-27T12:31:16.980000Z","created_at":"2024-07-27T12:31:16.980000Z"},{"_id":"66a4e894a6309140e005d44a","body":"Hey @awais786 & @UsamaSadiq, \r\n\r\nWhat is the status on this POC? Do we need to keep this PR open? It's been open for almost a year now and hasn't had any movement in 6 months.\r\n\r\nThanks!","issue_id":1667671331950,"origin_id":1602605026,"user_origin_id":3229735,"create_time":1687439094,"update_time":1687439094,"id":1722083476983,"updated_at":"2024-07-27T12:31:16.983000Z","created_at":"2024-07-27T12:31:16.983000Z"},{"_id":"66a4e894a6309140e005d44b","body":"In an effort to cleanup inactive PRs, I will be closing this PR. Please feel free to re-open if needed. I won't delete the branch either. ","issue_id":1667671331950,"origin_id":1650358105,"user_origin_id":3229735,"create_time":1690310855,"update_time":1690310875,"id":1722083476989,"updated_at":"2024-07-27T12:31:16.988000Z","created_at":"2024-07-27T12:31:16.988000Z"}]

### Description - POC PR to add `django-migration-linter` to test/inspect the migrations created in the PRs and identify the backwards incompatible migrations before they are merged to master.

awais786

docs: [FC-0002] plugin extensions ADR

[{"_id":"636f5c61b4ee765e3b109be3","body":"Thanks for the pull request, @GlugovGrGlib! Please note that it may take us up to several weeks or months to complete a review and merge your PR.\n\nFeel free to add as much of the following information to the ticket as you can:\n- supporting documentation\n- [Open edX discussion forum threads](https:\/\/discuss.openedx.org\/)\n- timeline information (\"this must be merged by XX date\", and why that is)\n- partner information (\"this is a course on edx.org\")\n- any other information that can help Product understand the context for the PR\n\nAll technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, [our process documentation is here](http:\/\/edx-developer-guide.readthedocs.org\/en\/latest\/process\/overview.html).\n\n\nPlease let us know once your PR is ready for our review and all tests are green.\n\n\n\n\n<!-- comment:external_pr -->\n<!-- jenkins ok to test -->\n<!-- data: eyJkcmFmdCI6IGZhbHNlfQ== -->\n","issue_id":1667671331954,"origin_id":1286153911,"user_origin_id":8964148,"create_time":1666300118,"update_time":1666300118,"id":1668242529435,"updated_at":"2022-11-12T08:42:09.435000Z","created_at":"2022-11-12T08:42:09.435000Z"},{"_id":"636f5c61b4ee765e3b109be4","body":"@GlugovGrGlib Thanks for this ADR, the references are very helpful, the decision seems well aligned with our other document architectural decisions. Broadly I approve of this direction, though I haven't officially approved the PR as I think it's critical that that is done by the maintaining team.","issue_id":1667671331954,"origin_id":1286972249,"user_origin_id":1270274,"create_time":1666359382,"update_time":1666359382,"id":1668242529439,"updated_at":"2022-11-12T08:42:09.439000Z","created_at":"2022-11-12T08:42:09.439000Z"},{"_id":"66a4e891616525d35f0c4661","body":"@cdeery @GlugovGrGlib cannot merge this, would you do the honors?","issue_id":1667671331954,"origin_id":1346676514,"user_origin_id":1270274,"create_time":1670857942,"update_time":1670857942,"id":1722083473176,"updated_at":"2024-07-27T12:31:13.175000Z","created_at":"2024-07-27T12:31:13.175000Z"},{"_id":"66a4e891616525d35f0c4662","body":"@GlugovGrGlib \ud83c\udf89 Your pull request was merged! Please take a moment to [answer a two question survey](https:\/\/docs.google.com\/forms\/d\/e\/1FAIpQLSceJOyGJ6JOzfy6lyR3T7EW_71OWUnNQXp68Fymsk3MkNoSDg\/viewform?usp=pp_url&entry.1671973413=openedx\/credentials&entry.867055334=https:\/\/github.com\/openedx\/credentials\/pull\/1797&entry.1484655318=https:\/\/github.com\/GlugovGrGlib&entry.752974735=2022-10-20+21:08&entry.1917517419=2022-12-12+19:51&entry.2133058324=Yes) so we can improve your experience in the future.\n\n<!-- comment:end_survey -->","issue_id":1667671331954,"origin_id":1347199192,"user_origin_id":8964148,"create_time":1670874674,"update_time":1670874674,"id":1722083473180,"updated_at":"2024-07-27T12:31:13.179000Z","created_at":"2024-07-27T12:31:13.179000Z"}]

In this ADR you will find a proposal to adopt plugin extensions system, initially implemented in the Open edX platform, but latter was extracted from it and now is ready...

GlugovGrGlib

Support Course Cert Signatory records in Credentials IDA

[{"_id":"636f5cf8a63f0c535f5fbb8e","body":"FYI @cdeery ","issue_id":1667671331958,"origin_id":1282960429,"user_origin_id":1400344,"create_time":1666124613,"update_time":1666124613,"id":1668242680389,"updated_at":"2022-11-12T08:44:40.388000Z","created_at":"2022-11-12T08:44:40.388000Z"},{"_id":"636f5cf8a63f0c535f5fbb8f","body":"@e0d Do we have a timeline for this? We may need a wiki page.","issue_id":1667671331958,"origin_id":1282962012,"user_origin_id":1400344,"create_time":1666124700,"update_time":1666360930,"id":1668242680392,"updated_at":"2022-11-12T08:44:40.391000Z","created_at":"2022-11-12T08:44:40.391000Z"},{"_id":"66a4e885616525d35f0c465e","body":"it's my belief that this epic and its associated tickets are no longer happening, is that correct? That is to say, the credentials IDA is _not_ the host of the LMS web certificates, and it is not going to become that host, and it should not be considered a blocker to any future deprecation of the old mongo store that has not been completed.\n\n Could interested parties tell me if I am mistaken about that anyway? Otherwise I will close this epic and the associated tickets.\n\ncc: @justinhynes @GlugovGrGlib","issue_id":1667671331958,"origin_id":2229191105,"user_origin_id":260800,"create_time":1721070284,"update_time":1721070284,"id":1722083461270,"updated_at":"2024-07-27T12:31:01.270000Z","created_at":"2024-07-27T12:31:01.270000Z"}]

- [ ] https://github.com/openedx/credentials/issues/1764 - [ ] https://github.com/openedx/credentials/issues/1765 - [ ] https://github.com/openedx/credentials/issues/1766

hurtstotouchfire

[DEPR]: Program Record UI

[{"_id":"636f5d89b4ee765e3b109cb9","body":"This frontned-app-learner-record MFE hasn't been included in an open release yet. I'm concerned about removing a feature in the same release that the MFE appears for the first time, i.e. olive. \r\n\r\nBut, that said, I'm not sure that the program-record UI has been supported in any prior open edX releases. \r\n","issue_id":1667671331961,"origin_id":1221046782,"user_origin_id":430126,"create_time":1660938984,"update_time":1660938984,"id":1668242825441,"updated_at":"2022-11-12T08:47:05.441000Z","created_at":"2022-11-12T08:47:05.441000Z"},{"_id":"636f5d89b4ee765e3b109cba","body":"@pdpinch yes, that's understandable, but the impression that I've gotten is that swapping them in one release should be possible and generally fails due to oversights -- and I'd love to pin down what those all are. It sounds like it's largely related to pain around configuration defaults?","issue_id":1667671331961,"origin_id":1238569193,"user_origin_id":1400344,"create_time":1662492759,"update_time":1662492759,"id":1668242825445,"updated_at":"2022-11-12T08:47:05.445000Z","created_at":"2022-11-12T08:47:05.445000Z"},{"_id":"636f5d89b4ee765e3b109cbb","body":"Sorry for the delayed response. BTR has an issue for documenting the requirements, https:\/\/github.com\/openedx\/build-test-release-wg\/issues\/131 but it doesn't seem complete yet. \r\n\r\nOff the top of my head, some of the hold-ups have been:\r\n\r\n- documentation for setup and configuration \r\n- support for translations \r\n- ability to deploy to a relative path, instead of to a separate named host (e.g. edx.org\/learn instead of learn.edx.org) \r\n- lingering edX references ","issue_id":1667671331961,"origin_id":1252873064,"user_origin_id":430126,"create_time":1663705502,"update_time":1663705502,"id":1668242825449,"updated_at":"2022-11-12T08:47:05.448000Z","created_at":"2022-11-12T08:47:05.448000Z"},{"_id":"636f5d89b4ee765e3b109cbc","body":"@arbrandes Are all DEPRs epics? If so that would save me having a dedicated view for them on https:\/\/github.com\/orgs\/openedx\/projects\/23\/","issue_id":1667671331961,"origin_id":1283027133,"user_origin_id":1400344,"create_time":1666128588,"update_time":1666128588,"id":1668242825451,"updated_at":"2022-11-12T08:47:05.451000Z","created_at":"2022-11-12T08:47:05.451000Z"},{"_id":"636f5d89b4ee765e3b109cbd","body":"@hurtstotouchfire, I tagged this one with `epic` for the benefit of the Frontend Development board, which has [a dedicated view](https:\/\/github.com\/orgs\/openedx\/projects\/29\/views\/2) for issues labelled as such.\r\n\r\nI'm of the opinion that, yes, any DEPR issue is by nature an epic, but I don't know if the DEPR working group feels the same way. Or even if they use the concept of epic at all. (I don't go to their meetings because it conflicts with the Contributors one, unfortunately.)","issue_id":1667671331961,"origin_id":1285575061,"user_origin_id":759355,"create_time":1666273625,"update_time":1666273625,"id":1668242825455,"updated_at":"2022-11-12T08:47:05.455000Z","created_at":"2022-11-12T08:47:05.455000Z"}]

### Proposal Date 19 aug 2022 ### Ticket Acceptance Date 25 sept 2022 ### Technology Removal Date 30 Oct 2022 (date subject to change but will be after Olive cutoff)...

hurtstotouchfire

Credentials IDA Tutor Support

[{"_id":"636f58131e434437e73ff691","body":"There is other plugin which I think is similar to the one you referred above: https:\/\/github.com\/lpm0073\/tutor-contrib-credentials ","issue_id":1667671331965,"origin_id":1282815937,"user_origin_id":16361375,"create_time":1666116822,"update_time":1666116822,"id":1668241427821,"updated_at":"2022-11-12T08:23:47.821000Z","created_at":"2022-11-12T08:23:47.821000Z"},{"_id":"66a4e8b9a6309140e005d451","body":"This was done at some point: https:\/\/github.com\/overhangio\/tutor-credentials\/","issue_id":1667671331965,"origin_id":2108008215,"user_origin_id":3628148,"create_time":1715614177,"update_time":1715614177,"id":1722083513123,"updated_at":"2024-07-27T12:31:53.123000Z","created_at":"2024-07-27T12:31:53.123000Z"}]

Currently, there is no official tutor-credentials plugin. There is a community one, but it is out of date and the docker image seems to be missing: https://github.com/pcliupc/tutor-credentials There is an...

hurtstotouchfire

Upgrade Credentials to Node 16

[]

- [ ] https://github.com/openedx/credentials/issues/1792 - [ ] Fix https://github.com/openedx/credentials-themes/issues/474

hurtstotouchfire

Switch from `node-sass` to `sass` library

[]

fix SASS related to print views for program certs which is breaking.

hurtstotouchfire

chore: update Credentials frontend dependencies to support Node16 upgrade

[]

[APER-2040]

justinhynes

build: Upgrading `django-simple-history`. It has some breaking changes.

[]

https://django-simple-history.readthedocs.io/en/latest/#section-1 Upgrading the package. https://github.com/jazzband/django-simple-history/pull/862 ( this pr shows the get_latest_by usage )

awais786

test: test invalid migration failure

[]

UsamaSadiq