Started working on removing last bits of frontend+backend monolith

Author: rhysyngsun

.env.example

@@ -85,7 +85,7 @@ jobs:
           CELERY_BROKER_URL: redis://localhost:6379/4
           CELERY_RESULT_BACKEND: redis://localhost:6379/4
           TIKA_CLIENT_ONLY: "True"
-          MITOPEN_BASE_URL: http://localhost:8063/
+          MITOPEN_APP_BASE_URL: http://localhost:8062/
           MAILGUN_KEY: fake_mailgun_key
           MAILGUN_SENDER_DOMAIN: other.fake.site
           OPENSEARCH_INDEX: testindex
@@ -123,7 +123,7 @@ jobs:
       - name: Webpack build
         run: yarn run build
         env:
-          MITOPEN_AXIOS_BASE_PATH: https://api.mitopen-test.odl.mit.edu
+          MITOPEN_API_BASE_URL: https://api.mitopen-test.odl.mit.edu
 
       - name: Lints
         run: yarn run lint-check

.github/workflows/ci.yml

@@ -36,7 +36,7 @@ jobs:
         env:
           EMBEDLY_KEY: ${{ secrets.EMBEDLY_KEY_PROD }}
           MITOPEN_AXIOS_WITH_CREDENTIALS: true
-          MITOPEN_AXIOS_BASE_PATH: https://api.mitopen.odl.mit.edu
+          MITOPEN_API_BASE_URL: https://api.mitopen.odl.mit.edu
 
       - uses: akhileshns/heroku-deploy@581dd286c962b6972d427fcf8980f60755c15520
         with:

.github/workflows/production.yml

@@ -36,7 +36,7 @@ jobs:
         env:
           EMBEDLY_KEY: ${{ secrets.EMBEDLY_KEY_RC }}
           MITOPEN_AXIOS_WITH_CREDENTIALS: true
-          MITOPEN_AXIOS_BASE_PATH: https://api.mitopen-rc.odl.mit.edu
+          MITOPEN_API_BASE_URL: https://api.mitopen-rc.odl.mit.edu
 
       - uses: akhileshns/heroku-deploy@581dd286c962b6972d427fcf8980f60755c15520
         with:

.github/workflows/release-candidate.yml

@@ -9,7 +9,6 @@ __pycache__/
 
 # Distribution / packaging
 .Python
-env/
 build/
 build-exports/
 develop-eggs/

.gitignore

@@ -25,11 +25,6 @@ The following settings must be configured before running the app:
 
   Controls which docker containers run. To run them all, use `COMPOSE_PROFILES=backend,frontend`. See [Frontend Development](./frontends/README.md) for more.
 
-- `INDEXING_API_USERNAME`
-
-  At least to start out, this should be set to the username of the superuser
-  you created above.
-
 - `MAILGUN_KEY` and `MAILGUN_SENDER_DOMAIN`
 
   You can set these values to any non-empty string value if email-sending functionality
@@ -216,7 +211,7 @@ Once these are set (and you've restarted the app), you should see events flowing
 A Javascript bundle of exported frontend components can be generated for use in external websites that have CORS allowance into a given instance of `mit-open`. There are a few settings you might want to change in order to get the expected results.
 
 - `MITOPEN_AXIOS_WITH_CREDENTIALS` - This sets `withCredentials: true` when initializing the Axios API, which tells the end user's browser to send along any browser level cookies for the current domain when making CORS requests
-- `MITOPEN_AXIOS_BASE_PATH` - This sets the base path used for API requests, which will need to be set to a fully qualified url pointing to an instance of `mit-open` (i.e. https://mitopen.odl.mit.edu) in order for requests from the external site to reach the proper destination
+- `MITOPEN_API_BASE_URL` - This sets the base url used for API requests, which will need to be set to a fully qualified url pointing to an instance of `mit-open` (i.e. https://mitopen.odl.mit.edu) in order for requests from the external site to reach the proper destination
 - `CORS_ALLOWED_ORIGINS`, `CSRF_TRUSTED_ORIGINS` - On the instance of `mit-open` that the externally hosted components will access via the API, the domains of any sites that need CORS access need to be here as a list of strings
 
 To build the bundle of exported components, run:

README.md

@@ -4,9 +4,6 @@
     {
       "url": "https://github.com/heroku/heroku-buildpack-apt"
     },
-    {
-      "url": "https://github.com/heroku/heroku-buildpack-nodejs"
-    },
     {
       "url": "https://github.com/moneymeets/python-poetry-buildpack"
     },
@@ -172,9 +169,6 @@
       "description": "Prefix path for cached images generated by imagekit",
       "required": false
     },
-    "INDEXING_API_USERNAME": {
-      "description": "Username used for indexing"
-    },
     "INDEXING_ERROR_RETRIES": {
       "description": "Number of times to retry an indexing operation on failure",
       "required": false
@@ -298,20 +292,12 @@
       "description": "List of pluggy plugins to use for authentication",
       "required": false
     },
-    "MITOPEN_AXIOS_WITH_CREDENTIALS": {
-      "description": "When building the Axios API, set defaults.withCredentials to this value",
-      "required": false
-    },
-    "MITOPEN_AXIOS_BASE_PATH": {
-      "description": "The base path to use when making API requests",
-      "required": false
-    },
     "MITOPEN_LEARNING_RESOURCES_PLUGINS": {
       "description": "List of pluggy plugins to use for learning resources",
       "required": false
     },
-    "MITOPEN_BASE_URL": {
-      "description": "Base url to link users to in emails"
+    "MITOPEN_APP_BASE_URL": {
+      "description": "Base url to create links to the app"
     },
     "MITOPEN_COOKIE_NAME": {
       "description": "Name of the cookie for the JWT auth token"

app.json

@@ -1,8 +1,5 @@
 """Models for channels"""
 
-from urllib.parse import urljoin
-
-from django.conf import settings
 from django.contrib.auth.models import Group
 from django.core.validators import RegexValidator
 from django.db import models
@@ -18,6 +15,7 @@
     LearningResourceTopic,
 )
 from main.models import TimestampedModel
+from main.utils import frontend_absolute_url
 from profiles.utils import avatar_uri, banner_uri
 from widgets.models import WidgetList
 
@@ -99,7 +97,7 @@ class Meta:
     @property
     def channel_url(self) -> str:
         """Return the channel url"""
-        return urljoin(settings.SITE_BASE_URL, f"/c/{self.channel_type}/{self.name}/")
+        return frontend_absolute_url(f"/c/{self.channel_type}/{self.name}/")
 
 
 class ChannelTopicDetail(TimestampedModel):

channels/models.py

@@ -1,10 +1,8 @@
 """Tests for channels.serializers"""
 
 from types import SimpleNamespace
-from urllib.parse import urljoin
 
 import pytest
-from django.conf import settings
 from django.core.files.uploadedfile import SimpleUploadedFile
 
 from channels.constants import CHANNEL_ROLE_MODERATORS, ChannelType
@@ -37,6 +35,7 @@
 )
 from learning_resources.serializers import LearningResourceOfferorDetailSerializer
 from main.factories import UserFactory
+from main.utils import frontend_absolute_url
 
 # pylint:disable=redefined-outer-name
 pytestmark = pytest.mark.django_db
@@ -124,8 +123,8 @@ def test_serialize_channel(  # pylint: disable=too-many-arguments
         "updated_on": mocker.ANY,
         "created_on": mocker.ANY,
         "id": channel.id,
-        "channel_url": urljoin(
-            settings.SITE_BASE_URL, f"/c/{channel.channel_type}/{channel.name}/"
+        "channel_url": frontend_absolute_url(
+            f"/c/{channel.channel_type}/{channel.name}/"
         ),
         "lists": [
             LearningPathPreviewSerializer(channel_list.channel_list).data