Skip to content

Commit

Permalink
Merge branch 'develop' into feature/#213_Create_orgs_pages
Browse files Browse the repository at this point in the history
  • Loading branch information
TwoSquirrels committed Sep 9, 2022
2 parents 638cc8e + 6a6f851 commit b1a87cd
Show file tree
Hide file tree
Showing 10 changed files with 69 additions and 42 deletions.
16 changes: 8 additions & 8 deletions DEVELOP.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@

### Branch

簡易的な [Git Flow](//dev.classmethod.jp/articles/introduce-git-flow/) で開発をします。
簡易的な [Git Flow](https://dev.classmethod.jp/articles/introduce-git-flow/) で開発をします。

- `main` : 本番用 / リリース用 (`develop` から merge する)
- `develop` : 開発版 (ここで直接は開発せず、 `feature/*` ブランチを切る。バージョンアップの時のみ直接コミットしてよい。)
Expand Down Expand Up @@ -36,20 +36,20 @@
### Lint

何らかの都合により一部分だけ lint を無効化したいときは、
ESLint の場合は [ESLint のコメントでのルール制御 - Qiita](//qiita.com/nju33/items/2d0cfea4fffbfdbff87a) の方法で、
textlint の場合は [textlint-filter-rule-comments](//github.com/textlint/textlint-filter-rule-comments) を入れているためその機能でできます。
また一部分だけ Prettier のフォーマットを無視させたい場合は [Ignoring Code - Prettier](//prettier.io/docs/en/ignore.html) の方法でできます。
ESLint の場合は [ESLint のコメントでのルール制御 - Qiita](https://qiita.com/nju33/items/2d0cfea4fffbfdbff87a) の方法で、
textlint の場合は [textlint-filter-rule-comments](https://github.com/textlint/textlint-filter-rule-comments) を入れているためその機能でできます。
また一部分だけ Prettier のフォーマットを無視させたい場合は [Ignoring Code - Prettier](https://prettier.io/docs/en/ignore.html) の方法でできます。

#### 「かた」と読む「方」が [ja-hiragana-keishikimeishi](//github.com/lostandfound/textlint-rule-ja-hiragana-keishikimeishi)`漢字よりもひらがなで表記したほうが読みやすい形式名詞` の「ほう」として判定されてしまう場合
#### 「かた」と読む「方」が [ja-hiragana-keishikimeishi](https://github.com/lostandfound/textlint-rule-ja-hiragana-keishikimeishi)`漢字よりもひらがなで表記したほうが読みやすい形式名詞` の「ほう」として判定されてしまう場合

このような場合 [textlint-filter-rule-comments](//github.com/textlint/textlint-filter-rule-comments) で無効化するのではなく、
このような場合 [textlint-filter-rule-comments](https://github.com/textlint/textlint-filter-rule-comments) で無効化するのではなく、
できるだけ「場合」や「人」など別の表現で置き換えましょう。

これは、「かた」として「方」を書いているつもりでも「ほう」と読み間違えてしまう可能性があるので、
それを防止するためです。

もちろん引用など、文章校正を受けたくない場合はコメントでのルール制御を行うべきですが、
そのような場合は [ja-hiragana-keishikimeishi](//github.com/lostandfound/textlint-rule-ja-hiragana-keishikimeishi) のみではなく
そのような場合は [ja-hiragana-keishikimeishi](https://github.com/lostandfound/textlint-rule-ja-hiragana-keishikimeishi) のみではなく
ルールを指定せずに全てのルールを無効化するべきです。

### Test
Expand Down Expand Up @@ -98,7 +98,7 @@ GitHub に push した際、この repository と連携している Cloudflare P

### バージョンアップ

1. [`develop` branch の preview](//develop.uchikoshi-fes-2022.pages.dev/) を確認する
1. [`develop` branch の preview](https://develop.uchikoshi-fes-2022.pages.dev/) を確認する
2. `develop` branch で `package.json``"version"` を更新し、
`README.md``Version:` を更新して `(括弧の中)` に主な更新点を記述。
3. `main` branch に Pull Request を投げる
Expand Down
14 changes: 7 additions & 7 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -26,29 +26,29 @@

**システム:**

- (全般) 文化祭実行委員 / 広報部門 / [TwoSquirrels](//github.com/TwoSquirrels)
- (全般) 文化祭実行委員 / 広報部門 / [TwoSquirrels](https://github.com/TwoSquirrels)
- (協力) 情報科の先生方
- (協力) [Mogyuchi さん](//github.com/Mogyuchi)
- (協力) [gendaineko2222 さん](//github.com/gendaineko2222)
- (協力) [Mogyuchi さん](https://github.com/Mogyuchi)
- (協力) [gendaineko2222 さん](https://github.com/gendaineko2222)

**デザイン:**

- (テーマ) 文化祭実行委員
- (コーディング) 文化祭実行委員 / 広報部門 / [TwoSquirrels](//github.com/TwoSquirrles)
- (コーディング) 文化祭実行委員 / 広報部門 / [TwoSquirrels](https://github.com/TwoSquirrles)

**コンテンツ:**

- (全般) 文化祭実行委員 / 広報部門

## Version

**Version:** 1.1.0 (予約フォーム追加)
**Latest Update:** 2022/09/05
**Version:** 1.1.1 (予約ページ調整)
**Latest Update:** 2022/09/09

## Feedback & Support

バグ報告や質問、提案があるなど、何か伝えたいことがある場合は、
GitHub アカウントをお持ちの場合は [Issues](//github.com/uchikoshi-fes/uchikoshi-fes.jp-2022/issues) に、
GitHub アカウントをお持ちの場合は [Issues](https://github.com/uchikoshi-fes/uchikoshi-fes.jp-2022/issues) に、
お持ちでない場合はお問い合わせフォーム (準備中) にお願いします。

## Tech Stack
Expand Down
6 changes: 5 additions & 1 deletion components/base/image.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,11 @@ const customLoader = ({ src }) => {
};

const Image = ({ ...props }) => {
return <NextImage {...props} loader={customLoader} />;
return process.env.isProduct ? (
<NextImage {...props} />
) : (
<NextImage {...props} loader={customLoader} />
);
};

export default Image;
2 changes: 1 addition & 1 deletion components/layout/header.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ import styles from "./header.module.scss";

const Header = ({ setScrollable }) => {
const isClient = useClient();
const isNarrow = useMediaQuery({ query: "(max-width: 600px)" });
const isNarrow = useMediaQuery({ query: "(max-width: 900px)" });

return (
<header className={styles.header}>
Expand Down
2 changes: 1 addition & 1 deletion components/layout/menu.module.scss
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,7 @@ $easeOutQuart: cubic-bezier(0.165, 0.84, 0.44, 1);

.wide {
height: 100%;
max-width: calc(100vw - 320px);
max-width: calc(100vw - 240px);
overflow-x: auto;

> ul {
Expand Down
21 changes: 18 additions & 3 deletions next.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,15 +4,30 @@
let config = {
reactStrictMode: true,
pageExtensions: ["js", "jsx", "md", "mdx"],
images: {
loader: "custom",
},
webpack: (config, { isServer }) => {
if (!isServer) config.resolve.fallback.fs = false;
return config;
},
};

config =
process.env.CF_PAGES_BRANCH === "main"
? {
...config,
env: { isProduct: true },
images: {
loader: "imgix",
path: "https://uchikoshi-fes-2022.imgix.net/",
},
}
: {
...config,
env: { isProduct: false },
images: {
loader: "custom",
},
};

config = require("@next/mdx")({
extension: /\.mdx?$/,
options: {
Expand Down
10 changes: 5 additions & 5 deletions package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "uchikoshi-fes.jp-2022",
"version": "1.1.0",
"version": "1.1.1",
"description": "第43回打越祭公式サイト",
"homepage": "https://github.com/uchikoshi-fes/uchikoshi-fes.jp-2022#readme",
"bugs": {
Expand Down Expand Up @@ -52,16 +52,16 @@
},
"devDependencies": {
"@testing-library/jest-dom": "5.16.5",
"@testing-library/react": "13.3.0",
"@testing-library/react": "13.4.0",
"@textlint-ja/textlint-rule-no-insert-dropping-sa": "2.0.1",
"@textlint/ast-node-types": "12.2.1",
"@textlint/types": "12.2.1",
"eslint": "8.23.0",
"eslint-config-next": "12.2.5",
"eslint-config-prettier": "8.5.0",
"husky": "8.0.1",
"jest": "29.0.1",
"jest-environment-jsdom": "29.0.1",
"jest": "29.0.2",
"jest-environment-jsdom": "29.0.2",
"lint-staged": "13.0.3",
"prettier": "2.7.1",
"textlint": "12.2.1",
Expand All @@ -84,6 +84,6 @@
"textlint-rule-preset-ja-spacing": "2.2.0",
"typescript": "4.8.2"
},
"packageManager": "pnpm@7.9.5",
"packageManager": "pnpm@7.11.0",
"private": true
}
2 changes: 1 addition & 1 deletion pages/_app.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ const UchikoshiFesApp = ({ Component, pageProps }) => {
React.useEffect(() => {
router.events.on("routeChangeComplete", resetScroll);
return () => router.events.off("routeChangeComplete", resetScroll);
}, []);
}, [router.events]);

return (
<>
Expand Down
14 changes: 11 additions & 3 deletions pages/reserve.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,9 @@ const NOTES = [
<li>
来場には抽選結果に記載される内容の提示が必要です。画面の提示または、プリントアウトのご準備をお願いします。
</li>
<li>
退場時間は設けておりません。 ※退場した後に再入場することはできません。
</li>
</ul>
),
},
Expand Down Expand Up @@ -71,12 +74,17 @@ const Reserve = () => {

return (
<>
<NextSeo title="来場申し込み" openGraph={{ title: "来場申し込み (抽選制)" }} />
<NextSeo
title="来場申し込み"
openGraph={{ title: "来場申し込み (抽選制)" }}
/>
<article className={styles.reserve}>
<h1>来場申込み (抽選制)</h1>
<h1>来場申し込み (抽選制)</h1>
{!process.env.NEXT_PUBLIC_RESERVE_URL && (
<article>
<h2 className={styles.warning}>現在申し込みは受け付けておりません</h2>
<h2 className={styles.warning}>
現在申し込みは受け付けておりません
</h2>
<p>2022/09/05 より申し込みを受け付けます。</p>
</article>
)}
Expand Down
24 changes: 12 additions & 12 deletions pnpm-lock.yaml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit b1a87cd

Please sign in to comment.