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

[feat] 年度別に取得するAPIを使って表示させるように変更 #688

Merged
merged 3 commits into from
Mar 4, 2024

Conversation

hikahana
Copy link
Collaborator

対応Issue

resolve #678

概要

前回、年度別に取得するAPIを作成したのでそれに伴ってフロントでフィルターを掛けるのではなくバックで取得したデータを表示させるように変更した

画面スクリーンショット等

  • URL
    スクリーンショット
    image

テスト項目

  • docker compose up -dでコンテナを立ち上げてlocalhost:3000/sponsorsにアクセスし、年度の切替でデータの取得、切替がなされているか確認お願いします。
  • consoleでsponsorsのurlを表示しているので、年度を変更した時に正しい年度のurlが確認できたら大丈夫だと思ってます。
  • 2023年度にはデータはないです。localhost:3000/yearperiodsで年度の編集ができるので2023年度の終了期間を変更すれば確認できると思います。

備考

過去の遺物に疑問が2点あります。

  • import { useState as UseState, useEffect as UseEffect } from 'react';でわざわざ別名を当てて定義しているのには意味があるのでしょうか。他のページではしてないようです。
  • onChange={(e) => setSelectedYear(e.target.value)}下記コードが他のページではonChange={async (e) => { setSelectedYear(e.target.value); }}とコーディングされています。syncを付けなくても動作するのですが何故つけているのでしょうか

@hikahana hikahana added enhancement New feature or request frontend labels Feb 27, 2024
@hikahana hikahana self-assigned this Feb 27, 2024
Copy link
Collaborator

@Kubosaka Kubosaka left a comment

Choose a reason for hiding this comment

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

動作問題なしです。
少しコードの修正お願いします。

@@ -1,42 +1,57 @@
import clsx from 'clsx';
import type { NextPage } from 'next';
import Head from 'next/head';
import { useState as UseState, useMemo as UseMemo } from 'react';

import { useState as UseState, useEffect as UseEffect } from 'react';
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
import { useState as UseState, useEffect as UseEffect } from 'react';
import { useState, useEffect } from 'react';

別名で定義する必要はないので修正してください。
これに伴うコードの修正もお願いします。

//年度別のsponsorsを取得
const getSponsors = async () => {
const getSponsorViewUrlByYear = process.env.CSR_API_URI + '/sponsors/' + selectedYear;
console.log(getSponsorViewUrlByYear);
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
console.log(getSponsorViewUrlByYear);

ログを消してください。

@Kubosaka Kubosaka self-requested a review March 4, 2024 04:40
Copy link
Collaborator

@Kubosaka Kubosaka left a comment

Choose a reason for hiding this comment

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

LGTM

@Kubosaka Kubosaka merged commit da0ad94 into develop Mar 4, 2024
@Kubosaka Kubosaka deleted the feat/hikahana/678-sponsors- branch March 4, 2024 04:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request frontend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

協賛企業ページの年度切り替え
2 participants