-
Notifications
You must be signed in to change notification settings - Fork 2
feat: support multisite switch via domain name #873
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
wesleyboar
wants to merge
5
commits into
main
Choose a base branch
from
feat/support-multisite
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
28 tasks
wesleyboar
added a commit
that referenced
this pull request
May 28, 2025
* refactor: CMD-97 breadcrumbs for news via 1 template * idea: TUP-706 4.a.i change RSS feed render⚠️ NOT WORKING! Fails with: > NoReverseMatch at /feed/ > '' is not a registered namespace * refactor: TUP-706 rename and document blog block * feat: TUP-706 render page with only (*_)content * fix: TUP-706 disable failed djangocms_blog_c… url * idea!: TUP-706 render blog base via custom app Fails, because it renders nothing. * feat!: TUP-706 render remote url content Load `/blog/remote`. BREAKING CHANGE: Changed `/blog` to `/blog/local` * chore: TUP-706 move an import higher in file * feat: TUP-706 render remote url via setting * fix: TUP-706 render blog customizations * fix: TUP-706 render feed⚠️ ⚠️ Does NOT render feed items. * docs: TUP-706 describe each URL's problem * docs: TUP-706 describe each URL goal and status * feat: TUP-706 A.1+3.c remote markup via template * feat: TUP-706 A.1.a. support BLOG_MULTISITE * fix: TUP-706 A.1.a. cannot login to site 2 * Revert "fix: TUP-706 A.1.a. cannot login to site 2" This reverts commit 6bece39. This did not fix anything. `SESSION_COOKIE_SECURE=False` is still required locally. * docs: TUP-706 feed status (ineffectual) * docs: tup-706 urls for local and remote * feat(poc): TUP-706 get news from remote url Messy but functional! * chore: remove test print statements from feeds * core: clean up views * core: clean up views more * refactor: switch order of methods in view class * feat: TUP-706 edit links via get_client_markup() * fix: TUP-706 broken links - ✓ fix links to articles - ✓ fix pagination links * chore: remove SITE_ID middleware I do not think it is a useful solution for TUP-706. The code/feature is archived in #873: #873 * chore: delete cruft from messy merge * refactor: djangocms_blog_custom… → remote_content This changes remote blog load to be remote anything load. This removes the failed custom feed (archived in #874): #874 * fix: load raw markup from remote site * fix: load raw markup from remote site * style: move parameter * feat!: change mage source not article link BREAKING CHANGE: images will not load, cuz of CORS. Solve via headers. * fix: links to articles should open on source * docs: how to test PR #868 * docs: fix how to test PR #868 * feat: (TEMPORARY) dynamic site id * docs: polish how to test PR #868 * fix: use settings for paths * feat: configurable template * refactor!: define template in URL not settings * refactor!: fix rel href paths, lose template param Gotta add back in support for tempalte param, and other CMS params. Working on it after this. * fix: tempalte param (and other cms params) KNOWN ISSUE: Frightening regex HTML manipulation. * docs: known issues * refactor: use beautiful soup not regex * docs: no major known issues * refactor: rename a function * chore; remove dumb comments * fix: support cms params in all href values * fix: do not try to load link rel=stylesheets * refactor: simplify href/src replacements * docs: comments * docs: how to convert (temp file) * feat!: use repo Core-CMS-Plugin-Remote-Content https://github.com/wesleyboar/Core-CMS-Plugin-Remote-Content BREAKING CHANGE: Repo has bug (see its issue #1) * refactor: replace internal app with external app * refactor: remove middleware * feat: update plugin * chore: delete outdated code Plugin handles this stuff now, and differently. * chore: update plugin (get improvements) * chore: update plugin (get better admin ui) * chore: update plugin (get better, refactored code) * test: djangocms_tacc_remote_content * chore: djangocms_tacc_remote_content via release * fix: remove extraneous dependency * fix: absolute urls for links by default * fix: keep relative urls via new settings * feat: do not passthrough cms query params * chore: v0.2.0 * refactor!: rename raw.html to bare.html Only pages testing this TUP-706 feature used `raw.html`. BREAKING CHANGE: Any pages using `raw.html` must switch to `bare.html` * chore: rename raw.html to bare.html in plugin * chore: rename bare.html to plain.html in plugin * chore: v0.3.0 * chore: TACC/Core-CMS-Plugin-Remote-Content * fix: rename bare.html file to plain.html * chore: v0.3.1 * chore: v0.3.2 * chore: v0.3.3 * feat: let admin change how to open external news
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Overview
Allow switching between multiple sites via domain name.
Related
Changes
Testing
custom_app_settings.py, add new middleware toCUSTOM_MIDDLEWAREe.g.SITEto one of those domain names e.g.0.0.0.0:80000.0.0.0:8000SITEwith the other domain names e.g.localhost:8000localhost:80000.0.0.0:8000.localhost:8000.Note
You may need to reload page after site/domain switch to refresh the CMS toolbar output.
UI
0.0.0.0:8000localhost:8000multisite.via.domain.mov