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

노트 수정 기능 추가랑 스타일링 (PR 뭉치기) #17

Closed
wants to merge 68 commits into from

Conversation

HotoRas
Copy link

@HotoRas HotoRas commented Sep 17, 2024

What

  • retrive old changes #14 (해당 PR의 내용은 여기에 뭉쳐졌습니다)
  • 기본 폰트 및 스킨셋 변경 (스킨은 설정에서 바꾸는 거와 별도로 소스 코드에도 합칠 예정
  • 글로벌 커스텀 css 적용
  • 프꾸 연합 기능 추가
  • 어딘가에 묻혀있던 냥체 번역 추가 (fall back to english maybe?)

Why

냥별의 색 입히기

Additional info (optional)

resolves #14

  • 스킨 추가
  • 기본 폰트 변경
  • 커스텀 css 적용
  • 프꾸 연합 yunochi#1
  • 한국어 냥체 추가

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests
  • (If possible) Publish docker image

HotoRas and others added 30 commits August 9, 2024 19:21
nekoplanet#13 Add changelog by @HotoRas
2215de5 by @HotoRas
c645e8f by @HotoRas
nekoplanet#15 노트 수정 기능 부활 (3트) by @HotoRas
nekoplanet#16 Feat: "다른 계정 추가" 버튼 아래에 "새 계정 추가" 버튼이 살아 있어서 지웠습니다 by @HotoRas
nekoplanet#17 Typecheck Fix by @janghoseo
nekoplanet#21 Fix note edit 2 by @HotoRas
- 중괄호(bracelets)가 제대로 닫히지 않아 코드 전체가 고장나는 이슈 수정
- scss 형식의 css 파트의 breaking changes 반영
Manual lint application to backend, bubble, reversi and sw
- Auto Lint by Lint Processor
회원가입이 아얘 버튼이 안 뜨게 되어있는데 버튼이 뜨는 상태에서만 테스트 가능한 부분이 있었음
IO용 패치에 대한 체인지로그 삭제
note.createdAt이 psql 쿼리 중 테이블 상에 없어서 오류 발생, 조치를 위해 마이그레이션 추가
syntax error
기존; notes/create api만 호출하도록 되어 있었음
-> notes/update와 이를 핸들링하기 위한 부분을 추가해 랜딩
and some effective changes for code
뭐 어떻게 하라는건데
and some pnpm.lock fixes
and some package version mismatch fixes
props.editMode가 true인 경우
- posting.value not in (null | undefined | false)
- posted.value not in (null | undefined | false)
인 상태라 별도로 해결해줘야 함
pnpm-lock.yaml이 변경되었습니다. 빌드 전 pnpm cleanall; pnpm i 명령을 꼭 실행해주세요.
체리픽을 했으니 어디서 했는진 알려줘야 될 거 아니여
기본 폰트를 오뮤 다예쁨체로 변경. 여기서 지원되지 않는 글리프는 기존 폰트 패밀리 (히라기노 마루 고딕, 비즈 유드고직, 로보토, 헬베티카 뉴, 아리얼, 시스템 산세리프)로 폴백

그리고 기존에 추가한 애 버그 잡았습니다
@HotoRas HotoRas marked this pull request as ready for review September 18, 2024 00:25
yunochi and others added 15 commits September 21, 2024 10:57
+ fix 7891331

+ fix !avatarDecorations
Resolves all type errors
## What
<!-- このPRで何をしたのか? どう変わるのか? -->
<!-- What did you do with this PR? How will it change things? -->
Merge yunochi#14 and more

## Why
<!-- なぜそうするのか? どういう意図なのか? 何が困っているのか? -->
<!-- Why do you do it? What are your intentions? What is the problem?
-->
Avatar Deco

## Additional info (optional)
<!-- テスト観点など -->
<!-- Test perspective, etc -->

## Checklist
- [x] Read the [contribution
guide](https://github.com/misskey-dev/misskey/blob/develop/CONTRIBUTING.md)
- [x] Test working in a local environment
- [ ] (If needed) Add story of storybook
- [x] (If needed) Update CHANGELOG.md
- [ ] (If possible) Add tests
@laonmofu
Copy link
Collaborator

기존의 기능들은 잘 작동합니다. 다만 작동이 안 되는 것들이 조금 있음.

  1. 스킨을 어디다 추가한건지 모르겠습니다. 어쨌든 수동으로 넣으니 만족스럽습니다.

  2. 기본 폰트 변경이 잘 되었습니다. 캐시 설정이랑 설정 중복이 있어서 약간 당황했어요.

  3. 커스텀 CSS 적용은 잘 되었으나 모든 맥락에서 너무 잠이 온다는 사실을 알리고 있습니다. (셀렉터 수정 예정)

스크린샷 2024-09-22 오후 5 25 15
  1. 프꾸 연합 잘 됩니다. 근데 이제 하나밖에 연합되지 않는.
스크린샷 2024-09-22 오후 5 27 28 5. 냥체 적용 잘 됩니다. 그리고 노트 수정도 기존처럼 잘 됩니다.

@HotoRas
Copy link
Author

HotoRas commented Sep 22, 2024

  1. 스킨을 어디다 추가한건지 모르겠습니다. 어쨌든 수동으로 넣으니 만족스럽습니다.

스킨은 현재 ...packages/frontend/src/themes/에 있습니다. 2024.9.0 업데이트로 스킨의 경로가 변경돼서 영향이 있을 거에요.

  1. 기본 폰트 변경이 잘 되었습니다. 캐시 설정이랑 설정 중복이 있어서 약간 당황했어요.

다행이네요.

  1. 커스텀 CSS 적용은 잘 되었으나 모든 맥락에서 너무 잠이 온다는 사실을 알리고 있습니다. (셀렉터 수정 예정)

셀렉터 말고 걍 vue 파일에 박죠?

  1. 프꾸 연합 잘 됩니다. 근데 이제 하나밖에 연합되지 않는.

으악 망했다

  1. 냥체 적용 잘 됩니다. 그리고 노트 수정도 기존처럼 잘 됩니다.

@HotoRas HotoRas mentioned this pull request Sep 30, 2024
6 tasks
@HotoRas HotoRas closed this in #19 Oct 20, 2024
@HotoRas HotoRas deleted the neko-feat/styling branch October 20, 2024 04:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

6 participants