Faeture/article update #39
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Collaborator
mucho7
commented
Aug 24, 2025
mucho7
commented
- 브랜치명과는 조금 다르게 delete는 버튼만 만들었습니다.
- update에 Github RAW파일 이슈가 있지만, 일단 업데이트가 완료되는 것까지는 구현이 되었습니다
- 실제로 업데이트가 완료됐지만, API로 접근할 수 있는 RAW파일의 경우 비교적 느리게 최신화되어 사용자의 화면엔 업데이트 전의 글이 보이는 이슈 가 있습니다
mucho7
force-pushed
the
faeture/article-updateDelete
branch
from
91ce128 to
2ff56bb
Compare
tevem1207
reviewed
Aug 24, 2025
Collaborator
There was a problem hiding this comment.
요거 생성페이지와 동일 코드 반복이 좀 많아서,
저번에 말씀드렸던 방식으로 처리하면 좋을 것 같습니다.
그것은 바로 create-post/1 처럼 게시글 id가 있을 경우 편집 페이지로 가는 방식
Collaborator
Author
There was a problem hiding this comment.
이거 그러면 url은 update-post로 유지하고 같은 컴포넌트를 바라보게 하는게 더 좋을라나요?
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.