Skip to content
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.

Sign up for GitHub

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

Jump to bottom

feat: tup-706 news on other sites #868

Draft
wants to merge 39 commits into
base: main
Choose a base branch
from
Draft

feat: tup-706 news on other sites #868

wants to merge 39 commits into from

Conversation

wesleyboar
Copy link
Member

@wesleyboar wesleyboar commented Sep 10, 2024 •
edited
Loading

Overview

Allow CMS News to be loaded on other websites.

Status

Tip

Loosely couple to blog feature:
  • Feature only expected for news.
  • H.P. says page other content should only live on one place.
  • I think code should be flexible.

Related

Changes

…

Testing

UI

…

wesleyboar and others added 20 commits July 19, 2024 15:54
@wesleyboar
refactor: CMD-97 breadcrumbs for news via 1 template
17640a4
@wesleyboar
idea: TUP-706 4.a.i change RSS feed render ⚠️
55dc431 
NOT WORKING! Fails with:

> NoReverseMatch at /feed/
> '' is not a registered namespace
@wesleyboar
refactor: TUP-706 rename and document blog block
de81256
@wesleyboar
feat: TUP-706 render page with only (*_)content
463871f
@wesleyboar
fix: TUP-706 disable failed djangocms_blog_c… url
8038de7
@wesleyboar
idea!: TUP-706 render blog base via custom app
89a160c 
Fails, because it renders nothing.
@wesleyboar
feat!: TUP-706 render remote url content
db234df 
Load `/blog/remote`.

BREAKING CHANGE: Changed `/blog` to `/blog/local`
@wesleyboar
chore: TUP-706 move an import higher in file
d7f10ae
@wesleyboar
feat: TUP-706 render remote url via setting
ec10c85
@wesleyboar
fix: TUP-706 render blog customizations
223b72a
@wesleyboar
Merge branch 'main' into feat/tup-706-news-on-other-sites
aaa971e
@wesleyboar
fix: TUP-706 render feed ⚠️
646b26d 
⚠️ Does NOT render feed items.
@wesleyboar
docs: TUP-706 describe each URL's problem
aa709fd
@wesleyboar
docs: TUP-706 describe each URL goal and status
26a9412
@wesleyboar
feat: TUP-706 A.1+3.c remote markup via template
4e78038
@wesleyboar
Merge branch 'main' into feat/tup-706-news-on-other-sites
17380fb
@wesleyboar
feat: TUP-706 A.1.a. support BLOG_MULTISITE
a461693
@wesleyboar
fix: TUP-706 A.1.a. cannot login to site 2
6bece39
@wesleyboar
Revert "fix: TUP-706 A.1.a. cannot login to site 2"
e6bdbbf 
This reverts commit 6bece39.

This did not fix anything. `SESSION_COOKIE_SECURE=False` is still required locally.
@wesleyboar
Merge branch 'main' into feat/tup-706-news-on-other-sites
4f9888b
@wesleyboar wesleyboar added feature A new feature or replacement of present feature effort XL Extra large effort task (~4 weeks) — does not include review time labels Sep 10, 2024
@wesleyboar wesleyboar force-pushed the feat/tup-706-news-on-other-sites branch from bd3397d to 104fe71 Compare September 10, 2024 22:37
This was referenced Sep 12, 2024
Draft
Merged
Open
@wesleyboar
Merge branch 'main' into feat/tup-706-news-on-other-sites
22c6ac3
@wesleyboar
chore: remove SITE_ID middleware
c1659f2 
I do not think it is a useful solution for TUP-706.

The code/feature is archived in #873:
#873
@wesleyboar wesleyboar force-pushed the feat/tup-706-news-on-other-sites branch from 86ecb25 to c1659f2 Compare September 13, 2024 21:33
@wesleyboar wesleyboar mentioned this pull request Sep 13, 2024
Closed
@wesleyboar
refactor: djangocms_blog_custom… → remote_content
7d56f8d 
This changes remote blog load to be remote anything load.

This removes the failed custom feed (archived in #874):
#874
@wesleyboar
Merge branch 'main' into feat/tup-706-news-on-other-sites
06304c9
@wesleyboar wesleyboar added the paused Started but not actively in progress label Sep 16, 2024
@wesleyboar wesleyboar mentioned this pull request Sep 17, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
effort XL Extra large effort task (~4 weeks) — does not include review time feature A new feature or replacement of present feature paused Started but not actively in progress
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@wesleyboar