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

Add link to study list #5

Merged
merged 5 commits into from
Nov 19, 2020

Conversation

saseungmin
Copy link
Collaborator

@saseungmin saseungmin commented Nov 18, 2020

  • 기존 a link CSS 수정하고 웹팩에 css-loader, style-loader 설정.
  • 스터디 목록의 제목에 Link추가.
  • 해당 스터디의 소개페이지로 이동하기.
  • 수정 사항 refactoring 하기 Refactorings received from code reviews #6

- npm install style-loader, css-loader, sass-loader
- Apply to index.jsx globally settings index.css
- Update test
- Improve Link CSS
- create redux dispatch and get api
- Add tests
- Fixed IntroduceContainer test
@saseungmin saseungmin added the enhancement New feature or request label Nov 18, 2020
Comment on lines +28 to +31
<IntroduceContainerWrapper>
<h2>{group.title}</h2>
<p>{group.contents}</p>
</IntroduceContainerWrapper>
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

  • 화면에 보여지는 부분을 더 추가하기
    이후 IntroduceComponent로 분리해주기.

@saseungmin
Copy link
Collaborator Author

  • 이번 PR을 훨씬 더 작게 생각하였다.
  • Component naming이나 변수 naming에 더 신경을 쓰면서 구현해야겠다.. 뭔가.. 마음에 안드는..

Comment on lines +5 to 6
import { MemoryRouter } from 'react-router-dom';
import StudyGroups from './StudyGroups';
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

  • import 한 칸 띄어써야지!
    또 신경 못 썼다. (자동완성의 폐해..)

Comment on lines +7 to 8
import { MemoryRouter } from 'react-router-dom';
import StudyGroupsContainer from './StudyGroupsContainer';
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

여기도.. 마찬가지

Comment on lines 5 to +6
import styled from '@emotion/styled';
import IntroduceContainer from '../containers/introduce/IntroduceContainer';
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

여기도....😢

@saseungmin saseungmin mentioned this pull request Nov 18, 2020
11 tasks
@saseungmin saseungmin merged commit aebaaa6 into CodeSoom:main Nov 19, 2020
@saseungmin saseungmin deleted the add_link_to_study_list branch July 5, 2021 19:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant