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.

Sign up for GitHub

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

Jump to bottom

🔀 :: [#384] 유관기관 기능 추가 #385

Merged
merged 4 commits into from
Jul 24, 2024
Merged

🔀 :: [#384] 유관기관 기능 추가 #385

merged 4 commits into from
Jul 24, 2024

Conversation

uuuunseo
Copy link
Contributor

@uuuunseo uuuunseo commented Jul 23, 2024 •
edited
Loading

💡 배경 및 개요

유관기관 페이지와, 전에 만든 기업 페이지가 모든 페이지 구성, 기능이 같고 다른 점은 "유관기관", "기업" 텍스트의 차이만 있어서 새로 유관기관 페이지를 만드는 건 비효율적이라고 생각했어요.

Resolves: #384

📃 작업내용

  • Organization이라는 기관, 기업 둘 다 포함하는 단어로 변경했어요. (Company -> Organization)
  • OrganizationType을 추가해 type에 맞는 페이지로 텍스트를 변경할 수 있도록 수정했어요.
  • 유관기관 리스트 조회, 유관기관 생성, 유관기관 삭제 기능을 추가했어요.

✅ PR 체크리스트

  • 이 작업으로 인해 변경이 필요한 문서가 변경되었나요? (e.g. .env, 노션, README)
  • 이 작업을 하고나서 공유해야할 팀원들에게 공유되었나요? (e.g. "API 개발 완료됐어요", "환경값 추가되었어요")
  • 작업한 코드가 정상적으로 동작하나요?
  • Merge 대상 브랜치가 올바른가요?
  • PR과 관련 없는 작업이 있지는 않나요?

@uuuunseo uuuunseo added 0️⃣ Priority: Critical 우선순위 - 긴급!!!!! ♻️ Refactor 코드 리팩토링 ⚡️performance 성능 개선 ✨ Feature 신규 기능 labels Jul 23, 2024
@uuuunseo uuuunseo self-assigned this Jul 23, 2024
@uuuunseo uuuunseo merged commit bf8af1f into master Jul 24, 2024
2 checks passed
@uuuunseo uuuunseo deleted the 384-expand-company-to-organization branch July 24, 2024 03:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@GangMinSu GangMinSu GangMinSu approved these changes

Assignees

@uuuunseo uuuunseo

Labels
0️⃣ Priority: Critical 우선순위 - 긴급!!!!! ✨ Feature 신규 기능 ⚡️performance 성능 개선 ♻️ Refactor 코드 리팩토링
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

유관기관 페이지
2 participants
@uuuunseo @GangMinSu