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

Fix resultpage firstbox #98

Open
wants to merge 9 commits into
base: dev
Choose a base branch
from
Open

Fix resultpage firstbox #98

wants to merge 9 commits into from

Conversation

yud0uhu
Copy link
Collaborator

@yud0uhu yud0uhu commented Apr 26, 2022

下記の改修をしました

  • 診断結果1位を示すボックス(firstCard)のUIを2位/3位(secondthirdCard)のデザインと統一
  • 順位を取得して表示する処理の実装
  • 変数名(_ありと_なしで混在していたところなど)を一部統一
  • rankingDataでバックエンドからすべての診断結果を取得し結果に反映させた

Mergeは#96 がMergeされ次第する感じでお願いします!

image

@yud0uhu yud0uhu marked this pull request as ready for review April 26, 2022 03:32
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.

2 participants