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(staff page)- update list #447

Merged
merged 2 commits into from
Aug 25, 2023
Merged

feat(staff page)- update list #447

merged 2 commits into from
Aug 25, 2023

Conversation

SivanYeh
Copy link
Collaborator

@SivanYeh SivanYeh commented Aug 19, 2023

Types of changes

  • Photo Update

Description

update new photo based on the list and drive

Steps to Test This Pull Request

Steps to reproduce the behavior:

  1. Type “/2023/en-us/about/staff” in url
  2. Scroll and check if the new list is correct
  3. Check again in “/2023/zh-hant/about/staff”

Expected behavior

image image

Related Issue

#438

Additional context

update_new_photo based on the list
@netlify
Copy link

netlify bot commented Aug 19, 2023

Deploy Preview for ephemeral-sable-89c7e0 ready!

Name Link
🔨 Latest commit 8fffc76
🔍 Latest deploy log https://app.netlify.com/sites/ephemeral-sable-89c7e0/deploys/64e70fc06af2a600085172f0
😎 Deploy Preview https://deploy-preview-447--ephemeral-sable-89c7e0.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@SivanYeh SivanYeh marked this pull request as ready for review August 20, 2023 07:42
Copy link
Collaborator

@baby230211 baby230211 left a comment

Choose a reason for hiding this comment

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

Nit: should we align the type of img? I have seen there are jpg, png...etc

teamName: 'Vice Chairperson',
},
{
key: 'everywhere',
Copy link
Member

Choose a reason for hiding this comment

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

  • 拿掉 everywhere 是有跟誰確認過嗎?如果確定要拿掉那 staff.vue 裡面的 array 也可以刪掉了
  • 為何只刪英文版部分?中文版卻留著?
  • 有新增 pycast 的照片但是這裡卻沒新增 pycast 的類別?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

謝謝你注意到了!
-everywhere: 以為圖庫跟大會Excel裡面都沒有這個類別就把他刪除了. 還是有什麼列入方式(重複出現3次就到everywhere, 或有固定班底)?
-pycast: 已新增

Copy link
Member

Choose a reason for hiding this comment

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

沒有固定規則,名單由當年度總招決定,可以把問題丟給 winnie & pochun

{ name: 'Po-Chun Lu', photo: 'cat-smile.png' },
{ name: 'Keith Yang', photo: 'reviewer/keith.jpg' },
{ name: 'Max Lai', photo: 'cat-smile.png' },
{ name: 'PeterWolf', photo: 'cat-smile.png' },
{ name: 'Ing Wei Tang', photo: 'reviewer/ingwei.jpg' },
{ name: '柯維然', photo: 'cat-smile.png' },
{ name: 'Winnie', photo: 'program/Winnie.jpeg' },
{ name: 'Jordan', photo: 'program/Jordan.jpg' },
{ name: 'Jordan', photo: 'cat-smile.png' },
Copy link
Member

Choose a reason for hiding this comment

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

這種明顯是同一人、但今年沒有上傳照片的,要不要直接使用舊照片?

Copy link
Contributor

Choose a reason for hiding this comment

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

同意,如果是同一人,可以用舊照片

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ok! 我試試

@yungshenglu
Copy link
Contributor

Nit: should we align the type of img? I have seen there are jpg, png...etc

光是照片大小,每個人都不一樣,格式我覺得還好,都顯得出來就好

Copy link
Contributor

@yungshenglu yungshenglu left a comment

Choose a reason for hiding this comment

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

應該修正完 Matt 的 Comment 就差不多了

@SivanYeh
Copy link
Collaborator Author

Nit: should we align the type of img? I have seen there are jpg, png...etc

光是照片大小,每個人都不一樣,格式我覺得還好,都顯得出來就好

Checked! I did it manually (WE'VE GOTTA FIND A BETTER WAY NEXT YEAR

@mattwang44
Copy link
Member

這真的是每年開發最繁雜無聊的頁面(對人名、收集照片、修照片 etc)XDD,真的辛苦了!!明年叫大家自己開 PR

幾個可以修正的 ideas

  • 叫大家設定 gravatar,只要有志工的 gmail 就可以拉到大頭照
  • 叫大家一定要去投稿系統辦帳號&設定大頭照,backend 開設新的 role for volunteer 並開放 get user API 喂給前端

Copy link
Contributor

@yungshenglu yungshenglu left a comment

Choose a reason for hiding this comment

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

LGTM

@yungshenglu yungshenglu merged commit bc2aa26 into main Aug 25, 2023
5 checks passed
@yungshenglu yungshenglu deleted the issue438-staff branch August 25, 2023 09:12
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.

4 participants