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.

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

「Devise で認証機能を追加」を更新して、"Logout" リンクを Rails 7 Turbo 向けに修正する #801

Merged
merged 1 commit into from
Mar 24, 2024

Conversation

eitoball
Copy link
Member

No description provided.

@urimaro
Copy link
Collaborator

urimaro commented Mar 21, 2024

@maimux2x
本家含めて確認する とのことでしたので、reviewerに設定しました 🙏
難しいようなら声かけてください

_pages/devise.md Outdated
<p id="notice"><%= notice %></p>
{% endhighlight %}

を削除します
というような行を削除します
Copy link
Collaborator

@maimux2x maimux2x Mar 21, 2024

Choose a reason for hiding this comment

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

細かいのですが、

というような

ではなく、「の行を削除します。」
の方がシンプルかと思いました。

Copy link
Member Author

@eitoball eitoball Mar 24, 2024

Choose a reason for hiding this comment

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

ありがとうございます。そうですね。そのように変更しました。

_pages/devise.md Outdated

## *5.*ユーザーを作成
## *5.* ユーザーを作成
Copy link
Collaborator

Choose a reason for hiding this comment

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

https://guides.railsgirls.com/devise
英語版の方だと「Create your first user」となっているため「first」の部分を追加した方が良さそうです。

Copy link
Member Author

Choose a reason for hiding this comment

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

ここは、入れようかどうしようか迷いました。「最初の」を入れても、あまり意味はないかなぁと思って、そのままにしておいたのですが、とりあえず、ここと本文の方も更新しました。

@maimux2x
Copy link
Collaborator

@eitoball
修正ありがとうございます🙏
少しコメントを入れたため、ご確認をいただけると幸いです。

@eitoball eitoball force-pushed the update-device-guide branch from 9fab2e3 to cdeddf7 Compare March 24, 2024 06:13
@maimux2x
Copy link
Collaborator

@eitoball
LGTMです🙆‍♀️
ありがとうございます〜!

@maimux2x maimux2x merged commit b3adc35 into master Mar 24, 2024
2 checks passed
@maimux2x maimux2x deleted the update-device-guide branch March 24, 2024 10:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants