Skip to content

[guides] Loginリンクをレイアウトに移動#1952

Merged
hachi8833 merged 1 commit intoyasslab:masterfrom
gskaeru:patch-1
Feb 3, 2026
Merged

[guides] Loginリンクをレイアウトに移動#1952
hachi8833 merged 1 commit intoyasslab:masterfrom
gskaeru:patch-1

Conversation

@gskaeru
Copy link

@gskaeru gskaeru commented Feb 3, 2026

概要

products/index ではなく layouts/application への記載が正しいかと思いました。ご確認頂けますと幸いです。

修正内容

オプションとして、先ほどのapp/views/layouts/application.html.erbレイアウトの<nav>セクションに以下のルーティングへのリンクも追加して、認証されていない場合はLoginリンクを表示するようにしてもよいでしょう。

<%# app/views/products/index.html.erb %>
<%= link_to "Login", new_session_path unless authenticated? %><%# app/views/layouts/application.html.erb %>
<%= link_to "Login", new_session_path unless authenticated? %>

@yasulab yasulab requested a review from hachi8833 February 3, 2026 03:40
@hachi8833
Copy link
Collaborator

プルリクありがとうございます!
<nav>はこのアプリのapplication.html.erbにしかないはずなので、原文も修正が必要そうですね。
先に私から原文にプルリク投げます。

@hachi8833
Copy link
Collaborator

@gskaeru 確認したところ、本家のmainブランチでも<%# app/views/layouts/application.html.erb %>に修正されていますね:

image

ただしmainにはそれ以外の変更もあるので、8-1-stableとの整合性を考えて、ここではいただいた修正をそのままマージすることにします。
ありがとうございます!

@hachi8833 hachi8833 merged commit 53c72c0 into yasslab:master Feb 3, 2026
4 checks passed
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.

2 participants