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 : showthreads complete #6

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

CHOSEUNGYEONN
Copy link

1. 본 PR이 우리 팀의 웹 서비스 제품성에 어떠한 기여를 하였고,
사용자에게 어떠한 기대효과를 전달하는지 작성해주세요.

  • 내 PR이 제품 내 어떠한 기능적인 배경/전후맥락 가운데 개발되었나요?

    쓰레드 기능에서 고객이 쓰레드 남긴 후 업데이트 순으로 현시될 수 있도록 개발 되었습니다.

  • 내 PR이 Merge 됨으로써 유저에게 전달되는 편익/기대효과는 무엇일까요?

    유저는 실시간으로 쓰레드를 남긴 후 가장 윗단에서 자신의 쓰레드를 확인할 수 있습니다.


2. 이 브랜치에서 어떤 내용을 개발했는지 큰 제목과 상세 내역을 적어주세요.

 데이터를 입력순이 아닌 역순으로 가져올 수 있게끔 개발하였습니다. 
:   SELECT 
    users.id, users.nickname, users.profile_image, threads.id, threads.content, threads.updated_at
    FROM threads 
    LEFT JOIN users
    ON threads.user_id = users.id
    ORDER BY 
    threads.updated_at DESC 

3. 개발한 화면을 캡쳐해서 첨부 해 주세요.


image

4. 이 브랜치에서 개발하면서 느꼇던 개발 성장포인트를 적어주세요.

  • 레이어드 패턴을 이해할 수 있는 시간이었습니다.
  • 데이터 정제하는 법을 익힐 수 있는 시간이었습니다.


const updateThread = async (req, res) => {
try {
const result = await thread.showThread(); // 괄호안에 키값
Copy link
Contributor

Choose a reason for hiding this comment

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

  • 상용화될 레포지토리인 main으로의 푸시 전 불필요한 주석문은 삭제하여주세요!

@@ -0,0 +1,17 @@
const { AppDataSource } = require("./dataSource");

const showThreads = async (/*매개변수(키값)*/) => {
Copy link
Contributor

Choose a reason for hiding this comment

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

  • 이곳에도 불필요한 의미없는 주석문이 들어있어 보입니다!

const showThreads = async (/*매개변수(키값)*/) => {
const test = await AppDataSource.query(
`SELECT
users.id, users.nickname, users.profile_image, threads.id, threads.content, threads.updated_at
Copy link
Contributor

Choose a reason for hiding this comment

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

  • 해당 내용은 가독성 향상을 위해, 각 컬럼 이후에 라인브레이킹을 진행하여 주세요!
SELECT 
   user.id,
   user.nickname... 이하 생략


const threadRouter = express.Router();

threadRouter.get('/show', threadController.updateThread);
Copy link
Contributor

Choose a reason for hiding this comment

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

  • RESTful API 엔드포인트 설계에 입각하여 동사형의 워딩을 엔드포인트에 작성해주시지 말고, 명사형태를 쓰거나, 동사형태는 http 메소드 설정으로 전가시켜주세요!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants