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

[Feat] 프로젝트 동적 라우팅 생성 #66

Merged
merged 1 commit into from
Nov 12, 2024
Merged

[Feat] 프로젝트 동적 라우팅 생성 #66

merged 1 commit into from
Nov 12, 2024

Conversation

iam454
Copy link
Collaborator

@iam454 iam454 commented Nov 12, 2024

관련 이슈 번호

#38

작업 내용

  • 프로젝트에 대한 동적 라우팅 적용

스크린샷

Screen.Recording.2024-11-12.at.7.32.41.PM.mov

함께 논의할 사항

프로젝트 이름은 영어로 제한하는 건 어떨까요?

프로젝트명이 uri로 노출되는 것이 좋을것 같다고 느껴서 현재 그렇게 적용된 상태인데요.
tanstack router에서 한글 uri에 대해 새로고침시 active 상태로 인식하지 않는 경우가 있더라고요?!
(이해가 안될것 같아서 영상으로 첨부합니다..)

Screen.Recording.2024-11-12.at.7.43.02.PM.mov

그래서 영어로 제한하자고 제안드려봅니다.
uri는 영어가 국룰이기도 하니까..
사실 uri 노출 안 시키면 되는 방법도 있긴 하는데..
근데 uri 노출시키니까 공백을 "-"로, "-"은 다시 공백으로 바꿔주는 작업이 늘어남

@iam454 iam454 added Feature 새로운 기능을 추가 FE 클라이언트 작업 labels Nov 12, 2024
@iam454 iam454 self-assigned this Nov 12, 2024
@PMtHk
Copy link
Collaborator

PMtHk commented Nov 12, 2024

프로젝트 이름은 영어로 제한하는 건 어떨까요?
프로젝트명이 uri로 노출되는 것이 좋을것 같다고 느껴서 현재 그렇게 적용된 상태인데요.
tanstack router에서 한글 uri에 대해 새로고침시 active 상태로 인식하지 않는 경우가 있더라고요?!
(이해가 안될것 같아서 영상으로 첨부합니다..)

저는 실제 url 에 노출되는 값은 project 의 name 값이 아닌 id 값(number or uuid) 이라고 생각하고 있습니다!
url 의 값과 사용자가 보는 값이 항상 동일할 필요는 없을 것 같아요~

@iam454
Copy link
Collaborator Author

iam454 commented Nov 12, 2024

그건 맞는데 이게 멋있지 않나요 ㅋㅋ

Copy link
Collaborator

@PMtHk PMtHk left a comment

Choose a reason for hiding this comment

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

👍🏻
좋습니다.

});

function RouteComponent() {
const { project } = Route.useParams();
Copy link
Collaborator

Choose a reason for hiding this comment

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

🟡
실제 프로젝트 값을 로딩한다면, project_id 의 값이 보여지는 것이 더 좋은 방향 같습니다.
(comment 로 작성해뒀습니다.)

@PMtHk
Copy link
Collaborator

PMtHk commented Nov 12, 2024

그건 맞는데 이게 멋있지 않나요 ㅋㅋ

github 이나 vercel 처럼 이름의 중복을 제한하면 가능하죠~
name 으로 primary key 처럼 사용하면 되겠네요!

사실 프론트에서 검증할 때 영어만 가능하다고 해버리면? ㅋㅋ

@Entity()
export class Project extends EntityTimestamp {
	@PrimaryGeneratedColumn()
	id: number;

	@Column()
	title: string;
}

@jjeonghak
Copy link
Collaborator

영어 이름으로 제한하는 것도 서버에서 가능합니다~

Copy link
Collaborator

@yangchef1 yangchef1 left a comment

Choose a reason for hiding this comment

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

ㅋㅋㅋㅋㅋ 멋있긴 한데 내가 소속되지 않은 프로젝트를 열람할 수 있는게 아닌데도 누군가 이미 쓰고 있다는 이유로 쓰고 싶은 이름을 못쓰면 살짝 화날거 같아요 😡

@yangchef1
Copy link
Collaborator

만약 구현한다고 하면, 검증은 클라이언트/서버 모두 적용하는게 좋을 것 같네용 ~

@PMtHk
Copy link
Collaborator

PMtHk commented Nov 12, 2024

만약 구현한다고 하면, 검증은 클라이언트/서버 모두 적용하는게 좋을 것 같네용 ~

좋아요! 로그인/회원가입은 적용 완료입니다!!

@PMtHk PMtHk merged commit 3219817 into dev Nov 12, 2024
2 checks passed
@iam454 iam454 deleted the feat/#38 branch November 13, 2024 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE 클라이언트 작업 Feature 새로운 기능을 추가
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants