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

ユーザー一覧のCSVダウンロード機能 #7

Merged
merged 2 commits into from
May 5, 2023

Conversation

MogamiTsuchikawa
Copy link
Member

@MogamiTsuchikawa MogamiTsuchikawa commented May 1, 2023

やったこと

/usersページに現在のユーザー一覧のCSVがダウンロードできるボタンを追加
image

クリックするとusers-現在の日付.csvがダウンロードされる

確認したこと

  • 複数ユーザーのデータで問題なく動作する
  • Excelで正しく表示される
  • ユーザー情報が取得できなかった際にクリックできない(無効化)

@ymd1138 ymd1138 marked this pull request as ready for review May 2, 2023 05:29
@ymd1138
Copy link
Collaborator

ymd1138 commented May 2, 2023

  • 選択している(表示されている)列に関係なく、すべての列をダウンロードしています。仕様はこれで大丈夫ですか。
  • 性別がtrue/falseになっているので、男/女に修正したい

@MogamiTsuchikawa
Copy link
Member Author

  • 選択している(表示されている)列に関係なく、すべての列をダウンロードしています。仕様はこれで大丈夫ですか。

CSVの場合、ここから加工する前提なので、問題無いと思います。
行フィルタは表示側含め今後の開発とします。

  • 性別がtrue/falseになっているので、男/女に修正したい

こちらも加工前提なので問題無いと考えています。(できるだけ判定すべきデータに日本語を入れたくない)

Copy link
Member

@Shimiiy Shimiiy left a comment

Choose a reason for hiding this comment

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

LGTM!

@MogamiTsuchikawa MogamiTsuchikawa merged commit 3ed7709 into master May 5, 2023
1 check 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.

None yet

3 participants