From 1fdc340bac6280b4cae0b78e1d758e443acdb207 Mon Sep 17 00:00:00 2001
From: Sarina Canelake <sarina@tcril.org>
Date: Sat, 10 Sep 2022 16:04:22 -0400
Subject: [PATCH] fix: fix github url strings (org edx -> openedx)
---
README.rst | 4 ++--
docs/decisions/0001-lti-extensions-plugin.rst | 6 +++---
docs/decisions/0002-lti-1p3-variables.rst | 2 +-
docs/decisions/0005-lti-pii-sharing-flag.rst | 6 +++---
lti_consumer/models.py | 2 +-
lti_consumer/plugin/compat.py | 2 +-
setup.py | 4 ++--
7 files changed, 13 insertions(+), 13 deletions(-)
diff --git a/README.rst b/README.rst
index 20202f5..40f4818 100644
--- a/README.rst
+++ b/README.rst
@@ -326,8 +326,8 @@ Reporting Security Issues
Please do not report security issues in public. Please email security@tcril.org.
-.. |ci-badge| image:: https://github.com/edx/xblock-lti-consumer/workflows/Python%20CI/badge.svg?branch=master
- :target: https://github.com/edx/xblock-lti-consumer/actions?query=workflow%3A%22Python+CI%22
+.. |ci-badge| image:: https://github.com/openedx/xblock-lti-consumer/workflows/Python%20CI/badge.svg?branch=master
+ :target: https://github.com/openedx/xblock-lti-consumer/actions?query=workflow%3A%22Python+CI%22
:alt: Test suite status
.. |codecov-badge| image:: https://codecov.io/github/edx/xblock-lti-consumer/coverage.svg?branch=master
diff --git a/docs/decisions/0001-lti-extensions-plugin.rst b/docs/decisions/0001-lti-extensions-plugin.rst
index 305d907..fcaa9eb 100644
--- a/docs/decisions/0001-lti-extensions-plugin.rst
+++ b/docs/decisions/0001-lti-extensions-plugin.rst
@@ -37,8 +37,8 @@ consumer to allow LTI launches from outside the context of an XBlock, following
While this is a smaller step than the implementation above, it allows LTI 1.1 launches on any LMS/Studio view, but doesn't implement
the configuration storage.
-.. _`Merge LTI 1.3 support to master`: https://github.com/edx/xblock-lti-consumer/pull/82
-.. _`Add support for LTI embeds in course tabs and elsewhere`: https://github.com/edx/xblock-lti-consumer/pull/77
+.. _`Merge LTI 1.3 support to master`: https://github.com/openedx/xblock-lti-consumer/pull/82
+.. _`Add support for LTI embeds in course tabs and elsewhere`: https://github.com/openedx/xblock-lti-consumer/pull/77
A few actions that need to be taken to achieve the desired architecture are outlined below:
@@ -93,7 +93,7 @@ Here is the new proposed folder structure::
(Here are the `Inter App APIs`_ guidelines.)
-.. _`Inter App APIs`: https://github.com/edx/edx-platform/blob/master/docs/decisions/0002-inter-app-apis.rst
+.. _`Inter App APIs`: https://github.com/openedx/edx-platform/blob/master/docs/decisions/0002-inter-app-apis.rst
Tech Debt
=========
diff --git a/docs/decisions/0002-lti-1p3-variables.rst b/docs/decisions/0002-lti-1p3-variables.rst
index b685953..a65ee07 100644
--- a/docs/decisions/0002-lti-1p3-variables.rst
+++ b/docs/decisions/0002-lti-1p3-variables.rst
@@ -106,7 +106,7 @@ The user data claims are identifiers passed through the LTI message to send user
* - sub
- external user id (with the `lti` type)
- Using the id provided by the external_user_id from openedx core.
- See https://github.com/edx/edx-platform/blob/master/openedx/core/djangoapps/external_user_ids/docs/decisions/0001-externalid.rst
+ See https://github.com/openedx/edx-platform/blob/master/openedx/core/djangoapps/external_user_ids/docs/decisions/0001-externalid.rst
* - name
- User full name
- User full name as defined in the spec, gated by a feature flag to avoid leaking PII to external tools.
diff --git a/docs/decisions/0005-lti-pii-sharing-flag.rst b/docs/decisions/0005-lti-pii-sharing-flag.rst
index ad18647..129cd48 100644
--- a/docs/decisions/0005-lti-pii-sharing-flag.rst
+++ b/docs/decisions/0005-lti-pii-sharing-flag.rst
@@ -14,15 +14,15 @@ about users, such as their username and email. Sharing such information allows
for a smoother registration flow among other benefits.
The `LTI XBlock has options
-<https://github.com/edx/xblock-lti-consumer/blob/edec2a68282a2a1fc2b2036e1989d60688fa6b19/lti_consumer/lti_xblock.py#L471-L487>`_
+<https://github.com/openedx/xblock-lti-consumer/blob/edec2a68282a2a1fc2b2036e1989d60688fa6b19/lti_consumer/lti_xblock.py#L471-L487>`_
called ``ask_to_send_username`` and ``ask_to_send_email`` that enable sending
such PII to the external tool. However, `by default these options are not
visible
-<https://github.com/edx/xblock-lti-consumer/blob/edec2a68282a2a1fc2b2036e1989d60688fa6b19/lti_consumer/lti_xblock.py#L588-L599>`_
+<https://github.com/openedx/xblock-lti-consumer/blob/edec2a68282a2a1fc2b2036e1989d60688fa6b19/lti_consumer/lti_xblock.py#L588-L599>`_
to course authors, and there is no way to send PII to LTI tools. In order to
even enable these options to appear in the LTI XBlock configuration, you need
to first set a `configuration flag
-<https://github.com/edx/edx-platform/blob/e19ba34f5a564285b3a20a7298c20ca640ca5aa0/cms/djangoapps/xblock_config/models.py#L38-L86>`_
+<https://github.com/openedx/edx-platform/blob/e19ba34f5a564285b3a20a7298c20ca640ca5aa0/cms/djangoapps/xblock_config/models.py#L38-L86>`_
via Django admin in studio.
This configuration flag unlocks the editing of the above fields after which
diff --git a/lti_consumer/models.py b/lti_consumer/models.py
index 2eb9ae0..1b83b0c 100644
--- a/lti_consumer/models.py
+++ b/lti_consumer/models.py
@@ -50,7 +50,7 @@ class LtiConfiguration(models.Model):
configuration data fetching them from XBlock fields.
With the implementation of
- https://github.com/edx/xblock-lti-consumer/blob/master/docs/decisions/0001-lti-extensions-plugin.rst
+ https://github.com/openedx/xblock-lti-consumer/blob/master/docs/decisions/0001-lti-extensions-plugin.rst
this model will store all LTI configuration values as a formatted JSON.
.. no_pii:
diff --git a/lti_consumer/plugin/compat.py b/lti_consumer/plugin/compat.py
index 5f65309..a15dae6 100644
--- a/lti_consumer/plugin/compat.py
+++ b/lti_consumer/plugin/compat.py
@@ -223,7 +223,7 @@ def get_course_by_id(course_key): # pragma: nocover
TODO: Once the LMS has fully switched over to this new path [1],
we can remove the legacy (LMS) import support here.
- - [1] https://github.com/edx/edx-platform/pull/27289
+ - [1] https://github.com/openedx/edx-platform/pull/27289
"""
# pylint: disable=import-outside-toplevel
try:
diff --git a/setup.py b/setup.py
index e46cb53..3ff3757 100644
--- a/setup.py
+++ b/setup.py
@@ -119,7 +119,7 @@ setup(
packages=find_packages(),
install_requires=load_requirements('requirements/base.in'),
dependency_links=[
- 'https://github.com/edx/xblock-utils/tarball/c39bf653e4f27fb3798662ef64cde99f57603f79#egg=xblock-utils',
+ 'https://github.com/openedx/xblock-utils/tarball/c39bf653e4f27fb3798662ef64cde99f57603f79#egg=xblock-utils',
],
entry_points={
'xblock.v1': [
@@ -134,7 +134,7 @@ setup(
},
package_data=package_data("lti_consumer", ["static", "templates", "public", "translations"]),
keywords='lti consumer xblock',
- url='https://github.com/edx/xblock-lti-consumer',
+ url='https://github.com/openedx/xblock-lti-consumer',
classifiers=[
'Development Status :: 5 - Production/Stable',
'Framework :: Django',
--
GitLab