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

Share extension과 영감 추가 분리 #529

Open
hyesungoh opened this issue Feb 28, 2023 · 0 comments
Open

Share extension과 영감 추가 분리 #529

hyesungoh opened this issue Feb 28, 2023 · 0 comments
Labels
refactor 코드 리팩토링

Comments

@hyesungoh
Copy link
Member

설명

같은 곳에서 두가지 기능을 대응하려다보니 로직의 분산도가 높아지는 것 같다고 생각돼요

As is

/add/{type}에서 두가지 방향을 대응하고 있어요

To be

add/image에서는 인앱 영감 추가만을 담당하고,

share/add/image (예)에서 share extension 로직을 담당하면 어떨까요?

코드의 중복이 되는 지점도 있겠지만, 로직을 분리하는 것이 추후 확장에 대응하기 쉬울 것 같다고 생각하는데 ..

제가 share extension에 대한 코드를 전부 읽어보진 않아서 아직까지는 그냥 추측입니다 !

@hyesungoh hyesungoh added the refactor 코드 리팩토링 label Feb 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor 코드 리팩토링
Projects
None yet
Development

No branches or pull requests

1 participant