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

[Bug]: author의 노션 계정에 프로필 사진이 없는 경우 [slug].tsx 페이지 접속시 에러가 발생합니다 #91

Closed
2 tasks done
itjustbong opened this issue Jan 27, 2023 · 2 comments
Labels
🐛bug Something isn't working

Comments

@itjustbong
Copy link
Contributor

Describe the bug

노션 계정의 사진이 없는 경우 하기와 같은 에러가 발생합니다.

스크린샷 2023-01-28 오전 4 56 18

위 에러는 노션 계정에 프로필 사진을 추가하니, 해결 되었습니다!
스크린샷 2023-01-28 오전 4 57 12

To reproduce

위에 관련 내용 작성하였습니다!

Browser

Google Chrome 109.0.5414.87 (공식 빌드) (arm64) and MacOS

Validations

@morethanmin morethanmin added the 🐛bug Something isn't working label Jan 29, 2023
@morethanmin
Copy link
Owner

이슈 감사합니다!
이부분은 이미지가 없을 때 예외처리를 해주거나 기본이미지를 보여줘야할 것 같은데, 혹시 의견 있으시면 고려해보고 없으시다면 이미지가 없을 경우에는 닉네임만 표시되는 형태로 수정할게요 👍

@itjustbong
Copy link
Contributor Author

빠른 확인 감사드립니다!

별 것 아닌 에러로 신경쓰실 것 같아,
PR요청했습니다.

노션 프로필 사진이 없을 경우에는 아바타 사진이 출력되도록 수정했습니다!
감사합니다!

itjustbong added a commit to itjustbong/morethan-log that referenced this issue Jan 31, 2023
morethanmin pushed a commit that referenced this issue Jan 31, 2023
* fix: no notion profile photo error (#91)

* chore: change profile image src path to const
gusleyva added a commit to gusleyva/c4cydonia-vercel-blog that referenced this issue Dec 17, 2023
* feat: update favicon and icon

* adding post author  (morethanmin#41)

* feat: update typescript

* fix: build error

* refactor: refactoring code

* refactor: refactoring code

* chore: update config

* chore: update README.md

* feat: updating author property

* Adding mobile profile section, Updating README (morethanmin#44)

* refactor: refactoring code

* refactor: refactoring code

* feat: adding responsive profile component

* fix: disable profile info

* feat: add bio and role

* chore: update README.md

* feat: mobile view

* feat: darkmode color

* chore: update config

* chore: update config

* Adding Post Custom Thumbnail (morethanmin#56)

* refactor: refactoring code

* refactor: refactoring code

* feat: adding responsive profile component

* fix: disable profile info

* feat: add bio and role

* chore: update README.md

* feat: mobile view

* feat: darkmode color

* fix: setting orderby variable

* feat: add thumbnail

* fix: build error

* fix: build error

* Adding Error Page, Fixing Some Issue (morethanmin#63)

* feat: add profile in post card component

* feat: style post header

* fix: updating profile_photo undefind check

* feat: adding 404 page

* feat: adding thumbnail in post detail page

* fix: using default next image loader

* fix: fixing user name issue

* Workflow Changes + Refactoring (morethanmin#67)

* feat: add prettier config file

* refactor: Move config files to site.config

* feat: Updated the Pull request template

* feat: Add Bug report Form

* feat: Updated the bug_report template

* feat: Updated code of conduct

* feat: Updated Contributing Docs

* feat: update Pull Request template

* fix: adding image domain (morethanmin#71)

* [Fix] notion-toggle 내부 element overflow 해결 (morethanmin#68)

* Bugfix: notion-toggle 내부 element overflow 해결

* Refactor: delete useless important

* [DOCUMENTATION] Documentation Updates (morethanmin#73)

* chore: move 2 files, update 2 files and delete 1 file

* docs: update PULL_REQUEST_TEMPLATE.md

* feat: update Footer.tsx

* feat: support mermaid digram  close morethanmin#77 (morethanmin#78)

* [REFACTOR] refactoring code (morethanmin#79)

* refactor: adding mermaid hook

* refactor: adding some type, refactoring get post

* chore: update config

* refactor: refactoring detail page

* feat: changing font (morethanmin#80)

* [UPDATE] tailwind.config.js (morethanmin#84)

* chore: update document

* chore: add contributors and sponsors

* docs: add contributors

* [FIX] no notion profile photo error (morethanmin#97)

* fix: no notion profile photo error (morethanmin#91)

* chore: change profile image src path to const

* docs: update contributors

* docs: update FUNDING.yml

* [FEAT] add cusdis comment support (morethanmin#103)

* feat: add cusdis feature

* docs: add cusdis to readme

* docs: update contributors

* fix: make cusdis theme same to blog theme (morethanmin#107)

* fix not applied fonts "Pretendard" (morethanmin#114)

* docs: update contributors

* typo: fix typo (morethanmin#120)

* docs: update contributors

* [FEAT] add "category" property (morethanmin#122)

* feat: add category

* feat: update coloring logic

* fix: update tailwind

* fix: add category readOnly props

* feat: add category inputs

* fix: category ui issue

* fix: typo

* [FIX] category overflow issue (morethanmin#123)

* feat: add category

* feat: update coloring logic

* fix: update tailwind

* fix: add category readOnly props

* feat: add category inputs

* fix: category ui issue

* fix: typo

* fix: navbar z-index

* fix: fix typo (TPostStatus) (morethanmin#132)

* fix: table notion overflow on mobile (morethanmin#144)

* docs: update contributors

* change to overflow auto (morethanmin#145)

* Fix:google analytics not working even though I have add ga id correctly (morethanmin#146)

* rename all env use in next build to NEXT_PUBLIC_

* fix document

* feat: ✨ adding sticky section (morethanmin#163)

* chore: 📦 update notion-client (morethanmin#171)

* [FEAT] react notion x 및 NotionRender 업데이트 (morethanmin#174)

* Feat: getPosts 요청 감소 로직 추가

* Feat: react-notion-x 업데이트 및 NotionRederer 수정

* build: 📦 update package

* feat: ✨ update background color

* [FIX] Notion Token Error (morethanmin#185)

* feat: add `index.d.ts` to declare `prismjs`

* feat: add code token recognition for multiple languages

* chore: add contributor

* chore: fix readme.md

* Update README.md (morethanmin#199)

Add user guide in english

* fix: fix serviceCard (morethanmin#238)

* chore: update readme.md

* [Feat] on demand revalidate api (morethanmin#224)

* feat: upgrade next, react, react-dom

* fix: remove a tag in link tag

- for next13

* chore: revalidateTime to confing

* feat: on demand revalidate api 추가

* doc: revalidate api 설명 주석 추가

* [FEAT] migrate emotion to tailwind (morethanmin#246)

* feat: update next.js

* feat: install emotion

* feat: convert layout style

* refactor: rename containers folder

* feat: convert category css

* feat: convert feed, taglist style

* feat: convert MobileProfileCard style

* feat: convert feed style

* feat: convert feed page style

* feat: ✨ remove tailwind

* feat: ✨ add themeProvider

* refactor: refactor structure

* refactor: refactor structure

* feat: darkmode

* feat: update theme

* fix: fix build error

* fix: fix post card ui (morethanmin#247)

* fix: fix comment darkmode (morethanmin#248)

* fix: fix tag background color

* fix: refactor comment box

* feat: improve notion css

* refactor: refactor error page

* refactor: refactor api

* refactor: ♻️ update meta config

* feat: refactoring ssr using react query

* fix: fix build error

* fix: remove console.log

* fix: set queryKey argument

* fix: fix default category not visible

* fix: fix type error

* feat: update feed ui

* feat: change utterances value

* FIX :: color set code to use record

* fix: change utterances value (morethanmin#271)

* docs: update readme.md

* docs: update README.md

* chore: update README.md

* feat: naver site verification

* Update site.config.js

* revert site.config.js

* clarification to README.md

* 20231217 - Download latest updates

---------

Co-authored-by: front_이상민 <[email protected]>
Co-authored-by: Lee Sang Min <[email protected]>
Co-authored-by: Vaishak Kaippanchery <[email protected]>
Co-authored-by: JaeSang1998 <[email protected]>
Co-authored-by: Obaid Al Teneiji <[email protected]>
Co-authored-by: Jinhwan Kim <[email protected]>
Co-authored-by: morethanmin <[email protected]>
Co-authored-by: itjustbong <[email protected]>
Co-authored-by: 도다 <[email protected]>
Co-authored-by: lisiver <[email protected]>
Co-authored-by: Octoping <[email protected]>
Co-authored-by: Octoping <[email protected]>
Co-authored-by: linnil <[email protected]>
Co-authored-by: 홍승현 <[email protected]>
Co-authored-by: halang <[email protected]>
Co-authored-by: SEOKKAMONI <[email protected]>
Co-authored-by: Randall <[email protected]>
Co-authored-by: Yongjun Cho <[email protected]>
Co-authored-by: Evelyn Gaskin <[email protected]>
Co-authored-by: Gustavo Leyva <[email protected]>
cheongsan pushed a commit to cheongsan/blog that referenced this issue Dec 12, 2024
* fix: no notion profile photo error (morethanmin#91)

* chore: change profile image src path to const
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants