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

replace: our University name #227

Merged
merged 3 commits into from
Oct 12, 2024
Merged

replace: our University name #227

merged 3 commits into from
Oct 12, 2024

Conversation

Pugma
Copy link
Collaborator

@Pugma Pugma commented Oct 12, 2024

User description

大学統合に伴う名称変更に対応しました


PR Type

enhancement


Description

  • 大学名を「東京工業大学」から「東京科学大学」に変更しました。
  • src/pages/IndexPage.vueindex.htmlの説明文を更新しました。
  • ポートフォリオサービスのメタデータを修正しました。

Changes walkthrough 📝

Relevant files
Enhancement
IndexPage.vue
大学名の変更に伴うポートフォリオ説明の更新                                                                       

src/pages/IndexPage.vue

  • 大学名を「東京工業大学」から「東京科学大学」に変更
  • ポートフォリオの説明文を更新
+2/-2     
index.html
大学名の変更に伴うメタデータの更新                                                                               

index.html

  • 大学名を「東京工業大学」から「東京科学大学」に変更
  • メタデータの説明文を更新
+3/-3     

💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

Copy link

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

Copy link

PR Code Suggestions ✨

Explore these optional code suggestions:

CategorySuggestion                                                                                                                                    Score
Best practice
HTMLの<br />タグを<br>に修正して、HTML5標準に合わせます。

HTML内の<br />タグを<br>に変更してください。XHTML以外では自己終了タグは推奨されません。

src/pages/IndexPage.vue [48]

-東京科学大学・大岡山キャンパスを拠点に活動する創作・プログラミングの総合サークルです。<br />
+東京科学大学・大岡山キャンパスを拠点に活動する創作・プログラミングの総合サークルです。<br>
Suggestion importance[1-10]: 7

Why: The suggestion to change <br /> to <br> aligns with HTML5 standards, which do not require self-closing tags. This improves code consistency and adheres to modern best practices.

7
Enhancement
メタタグの重複を解消して、HTMLの冗長性を減らします。

同じ内容のメタタグのcontent属性が重複しています。重複を避けるために、一つのメタタグに統合することを検討してください。

index.html [26-27]

-<meta name="twitter:description" content="東京科学大学デジタル創作同好会traPに所属する各部員の活動内容をまとめたポートフォリオサービスです。">
 <meta name="description" content="東京科学大学デジタル創作同好会traPに所属する各部員の活動内容をまとめたポートフォリオサービスです。" />
Suggestion importance[1-10]: 6

Why: The suggestion to remove the duplicate meta tag reduces redundancy and potential confusion in the HTML document. However, both tags serve different purposes, so the suggestion should be carefully considered before implementation.

6

Copy link
Member

@ras0q ras0q left a comment

Choose a reason for hiding this comment

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

ここは別PRでやります?
そこ以外はよさそうです:+1:
image

@Pugma
Copy link
Collaborator Author

Pugma commented Oct 12, 2024

僕の手元だとフッターは Futadaruma くんの PR マージによって差し替わったあとで表示されてますね

@ras0q
Copy link
Member

ras0q commented Oct 12, 2024

(c) 2015 - traP TokyoTechが変わってなさそう?

@Pugma
Copy link
Collaborator Author

Pugma commented Oct 12, 2024

あ、忘れてたのでやります

@Futadaruma
Copy link
Contributor

フッターの変更ありがとうございます!!
良さそうです!

一つ気になることがあるとすると、trap.jpでは© 2015-2024 traPとなっていてtraPortfolioは© 2015- traPになっているから統一した方が良いのかな〜?って思ってます
(個人的には、毎年の更新が不要な後者の方が好みです)

@Pugma
Copy link
Collaborator Author

Pugma commented Oct 12, 2024

フッターの変更ありがとうございます!! 良さそうです!

一つ気になることがあるとすると、trap.jpでは© 2015-2024 traPとなっていてtraPortfolioは© 2015- traPになっているから統一した方が良いのかな〜?って思ってます (個人的には、毎年の更新が不要な後者の方が好みです)

これに関しては前の PR で現在の年を省くことになっていたのでそれに合わせてます
ズレはあるけど問題ない、という認識だったような気がするので大丈夫だと思います!

@Pugma Pugma merged commit 092d6e4 into master Oct 12, 2024
6 checks passed
@Pugma Pugma deleted the replaceUnivName branch October 12, 2024 22:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants