Skip to content

feat: offer client user in nav_portal.html#1115

Merged
wesleyboar merged 2 commits intomainfrom
feat/allow-client-to-access-user
Mar 11, 2026
Merged

feat: offer client user in nav_portal.html#1115
wesleyboar merged 2 commits intomainfrom
feat/allow-client-to-access-user

Conversation

@wesleyboar
Copy link
Member

@wesleyboar wesleyboar commented Mar 11, 2026

Overview

Let client (e.g. TACC/tup-ui) access user in nav_portal.html.

Related

This will be required by TACC/tup-ui after it uses —

— and otherwise makes sense for portal nav to access user.

Changes

  • adds user to limited context

Testing / UI

  1. See refactor: simplify portal nav tup-ui#541

@wesleyboar wesleyboar marked this pull request as ready for review March 11, 2026 23:05
@wesleyboar wesleyboar merged commit 7a49f6c into main Mar 11, 2026
@wesleyboar wesleyboar deleted the feat/allow-client-to-access-user branch March 11, 2026 23:05
@qodo-code-review
Copy link

Review Summary by Qodo

Expose user context to nav_portal.html template

✨ Enhancement

Grey Divider

Walkthroughs

Description
• Expose user context variable to nav_portal.html template
• Enables client applications to access user data in portal navigation
• Required for TACC/tup-ui integration with Core-CMS v4.35.3+
Diagram
flowchart LR
  header["header.html"] -- "passes user parameter" --> nav_portal["nav_portal.html"]
  nav_portal -- "can now access" --> user["user context"]
Loading

Grey Divider

File Changes

1. taccsite_cms/templates/header.html ✨ Enhancement +1/-1

Pass user context to nav_portal.html include

• Added user=user parameter to nav_portal.html template include statement
• Allows portal navigation template to access user context variable
• Maintains existing only constraint for template variable isolation

taccsite_cms/templates/header.html


Grey Divider

Qodo Logo

@qodo-code-review
Copy link

qodo-code-review bot commented Mar 11, 2026

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

ⓘ The new review experience is currently in Beta. Learn more

Grey Divider

Qodo Logo

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