Skip to content

feat: auto-manage external articles#1033

Merged
wesleyboar merged 6 commits into
stablefrom
feat/auto-handle-external-articles
Nov 18, 2025
Merged

feat: auto-manage external articles#1033
wesleyboar merged 6 commits into
stablefrom
feat/auto-handle-external-articles

Conversation

@wesleyboar

@wesleyboar wesleyboar commented Nov 18, 2025

Copy link
Copy Markdown
Member

Overview

Auto-handle external articles via Blog/News app. Client app does nothing.

The client can still use #1032's script, but should not do so after this.

Related

Changes

  • added external article management into news template
  • changed setting default to support this

Testing

  1. Have a blog/news set up.
  2. Create a blog/news article.
  3. Tag article as external.
  4. In article abstract, add link to external website.
  5. In blog/news list, click article.
  6. Verify external website opens.
  7. In blog/news, open article directly.
  8. Verify article redirects to external website.
  9. In blog/news, open article directly but with ?edit on URL.
  10. Verify article does not redirect to external website.

UI

News.list.changes.news.link.ot.external.URL.mov
Visiting.article.redirects.to.external.site.mov
Visiting.article.in.edit.mode.does.not.redirect.article.mov

@wesleyboar wesleyboar left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Notes

Comment thread taccsite_cms/templates/djangocms_blog/post_detail.html
Comment thread taccsite_cms/templates/djangocms_blog/post_list.html
Comment thread taccsite_cms/settings.py
@wesleyboar
wesleyboar marked this pull request as ready for review November 18, 2025 22:13
@wesleyboar
wesleyboar merged commit a009a85 into stable Nov 18, 2025
@wesleyboar
wesleyboar deleted the feat/auto-handle-external-articles branch November 18, 2025 22:14
wesleyboar added a commit to TACC/tup-ui that referenced this pull request Feb 5, 2026
TACC/Core-CMS#1033 automates redirect logic.

#529 was stepping stone.

But this PR does not need those steps nor old code.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant