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

Feature/#355 - 검색어 빈 값 요청 처리, 초기 로딩 시 그래프만 다크모드인 문제 수정 #357

Merged
merged 2 commits into from
Dec 4, 2024

Conversation

baegyeong
Copy link
Collaborator

close #355 #354

✅ 작업 내용

  • 검색어가 빈 값 일 경우 요청하지 못하도록 수정
  • 초기 로딩 시 그래프만 다크모드로 설정되는 문제 수정

😎 체크 사항

  • label 설정 확인
  • 브랜치 방향 확인

@@ -15,6 +15,7 @@ export const Search = ({ className }: SearchProps) => {

const handleSubmit = (event: FormEvent<HTMLFormElement>) => {
event.preventDefault();
if (!stockName) return;
Copy link
Collaborator

Choose a reason for hiding this comment

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

서버쪽에도 막아야하는데... 한번 해볼게요!

Copy link
Collaborator

@swkim12345 swkim12345 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

Copy link
Collaborator

@xjfcnfw3 xjfcnfw3 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다! 👍

@@ -44,7 +44,11 @@ export const useChart = ({
const containerInstance = containerRef.current;

const { theme } = useContext(ThemeContext);
const graphTheme = theme === 'light' ? lightTheme : darkTheme;
const graphTheme = theme
? theme === 'light'
Copy link
Collaborator

Choose a reason for hiding this comment

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

?를 연속해서 쓸 수 있군요

@baegyeong baegyeong merged commit 513c91b into dev-fe Dec 4, 2024
@swkim12345 swkim12345 deleted the feature/#355 branch December 5, 2024 02:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE 🐞 bugfix Something isn't working
Projects
None yet
3 participants