Skip to content

Fix PreviousMenuNext on Django Forms page#44038

Open
caugner wants to merge 2 commits intomainfrom
django-forms-previousmenunext
Open

Fix PreviousMenuNext on Django Forms page#44038
caugner wants to merge 2 commits intomainfrom
django-forms-previousmenunext

Conversation

@caugner
Copy link
Copy Markdown
Collaborator

@caugner caugner commented May 7, 2026

Description

Fixes PreviousMenuNext parameter on Django Forms page.

Motivation

Resolve a Rari issue that was propagated to other locales.

Additional details

The top/bottom invocations were out of sync, with
the bottom one pointing to a non-existing page.

Related issues and pull requests

Part of mdn/fred#1462.

The top/bottom invocations were out of sync, with
the bottom one pointing to a non-existing page.
@caugner caugner requested a review from a team as a code owner May 7, 2026 19:53
@caugner caugner requested review from chrisdavidmills and removed request for a team May 7, 2026 19:53
@github-actions github-actions Bot added Content:Learn Learning area docs size/xs [PR only] 0-5 LoC changed labels May 7, 2026
Comment thread files/en-us/learn_web_development/extensions/server-side/django/forms/index.md Outdated
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 7, 2026

Preview URLs (1 page)

(comment last updated: 2026-05-07 20:28:30)

…o/forms/index.md

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
- [Generic editing views](https://docs.djangoproject.com/en/5.0/ref/class-based-views/generic-editing/) (Django docs)

{{PreviousMenuNext("Learn_web_development/Extensions/Server-side/Django/authentication_and_sessions", "Learn_web_development/Extensions/Server-side/Django/Testing", "Learn_web_development/Extensions/Server-side/Django")}}
{{PreviousMenuNext("Learn_web_development/Extensions/Server-side/Django/Sessions", "Learn_web_development/Extensions/Server-side/Django/Testing", "Learn_web_development/Extensions/Server-side/Django")}}
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I think this should be authentication, just with a different URL

Suggested change
{{PreviousMenuNext("Learn_web_development/Extensions/Server-side/Django/Sessions", "Learn_web_development/Extensions/Server-side/Django/Testing", "Learn_web_development/Extensions/Server-side/Django")}}
{{PreviousMenuNext("/Learn_web_development/Extensions/Server-side/Django/Authentication", "Learn_web_development/Extensions/Server-side/Django/Testing", "Learn_web_development/Extensions/Server-side/Django")}}

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Note that I "just" copied down the line from the top, so if we apply this change, it needs to be replicated further up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:Learn Learning area docs size/xs [PR only] 0-5 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants