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

[수정] 검색기록 최신순으로 저장되도록 변경 #233

Merged
merged 5 commits into from
Sep 11, 2024

Conversation

sunshiningsoo
Copy link
Member

  • 디자인팀에서 '검색기록 최신순일수록 왼쪽에 배치'로 말씀하셔서 해당기능 수정합니답
2024-09-09.19.41.14.mov

@sunshiningsoo sunshiningsoo added the 새기능/개선작업 새 기능 작업 또는 개선 작업 label Sep 9, 2024
@sunshiningsoo sunshiningsoo self-assigned this Sep 9, 2024
@sunshiningsoo
Copy link
Member Author

/쿠링 패키지 빌드

Copy link

github-actions bot commented Sep 9, 2024

🔨 iOS 17.0.1 iPhone 15 Pro 에서 스위프트 패키지를 빌드합니다.

Copy link

github-actions bot commented Sep 9, 2024

📬 패키지 빌드에 실패했습니다. Actions에서 실패 결과를 확인해주세요.

@lgvv
Copy link
Member

lgvv commented Sep 9, 2024

기존에 검색어 하나하나씩 삭제하는 있는걸로 기억하는데, 혹시 이번에 빠지는게 맞을까요?

@sunshiningsoo
Copy link
Member Author

기존에 검색어 하나하나씩 삭제하는 있는걸로 기억하는데, 혹시 이번에 빠지는게 맞을까요?

그 기능 없는거 같아요??!

@lgvv
Copy link
Member

lgvv commented Sep 9, 2024

기존에 검색어 하나하나씩 삭제하는 있는걸로 기억하는데, 혹시 이번에 빠지는게 맞을까요?

그 기능 없는거 같아요??!

v1 있던걸로 기억하는데, v2에서 빠진 기능이었나봐요!!

@sunshiningsoo
Copy link
Member Author

v1 있던걸로 기억하는데, v2에서 빠진 기능이었나봐요!!

이거는 투두에 추가해두겠습니닷! 빌드 오류나는게 어떤이유인지 헲 부탁드립니다..ㅎ 정확히 모르겠네요

lgvv
lgvv previously approved these changes Sep 9, 2024
Copy link
Member

@lgvv lgvv left a comment

Choose a reason for hiding this comment

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

insert 맨 앞자리에 하면 모든 배열을 미뤄야 할 것으로 생각되는데, 시간 복잡도 고려하면 append로 처리하고, UI 그리는 부분에서 reversed() 처리할 수도 있을 것 같아요!

@lgvv
Copy link
Member

lgvv commented Sep 9, 2024

v1 있던걸로 기억하는데, v2에서 빠진 기능이었나봐요!!

이거는 투두에 추가해두겠습니닷! 빌드 오류나는게 어떤이유인지 헲 부탁드립니다..ㅎ 정확히 모르겠네요

ㅠㅠ 야근이라 오늘 집에가면 1-2시 쯤 될 것 같은데, CI 관련해서 이번주 중으로 에너지 끌어 모아서 꼭 처리해두도록 할게요!

@lgvv
Copy link
Member

lgvv commented Sep 9, 2024

2024-09-09T10:50:05.6927750Z 
2024-09-09T10:50:05.9896850Z ** BUILD FAILED **
2024-09-09T10:50:05.9914490Z 
2024-09-09T10:50:05.9915580Z 
2024-09-09T10:50:05.9917020Z The following build commands failed:
2024-09-09T10:50:05.9920110Z 	SwiftEmitModule normal x86_64 Emitting\ module\ for\ CampusUI (in target 'CampusUI' from project 'package-kuring')
2024-09-09T10:50:05.9922850Z (1 failure)
2024-09-09T10:50:06.0451080Z ##[error]Process completed with exit code 65.
2024-09-09T10:50:06.0983140Z ##[group]Run actions/github-script@v5
2024-09-09T10:50:06.0983690Z with:
2024-09-09T10:50:06.0984340Z   github-token: ***
2024-09-09T10:50:06.0985950Z   script: github.rest.issues.createComment({
  issue_number: context.issue.number,
  owner: context.repo.owner,
  repo: context.repo.repo,
  body: '📬 패키지 빌드에 실패했습니다. Actions에서 실패 결과를 확인해주세요.'
})

@sunshiningsoo
Copy link
Member Author

sunshiningsoo commented Sep 10, 2024

insert 맨 앞자리에 하면 모든 배열을 미뤄야 할 것으로 생각되는데, 시간 복잡도 고려하면 append로 처리하고, UI 그리는 부분에서 reversed() 처리할 수도 있을 것 같아요!

append 사용하는편이 훨 좋을 것 같습니다!! 감사합니다~!
추가로,, CI 천천히 봐주셔도 괜찮아서 새벽에는 쉬십셔,,ㅎ
adffb55 수정완!

lgvv
lgvv previously approved these changes Sep 10, 2024
@sunshiningsoo
Copy link
Member Author

/쿠링 패키지 빌드

Copy link

🔨 iOS 17.0.1 iPhone 15 Pro 에서 스위프트 패키지를 빌드합니다.

Copy link

📬 패키지 빌드에 실패했습니다. Actions에서 실패 결과를 확인해주세요.

@sunshiningsoo
Copy link
Member Author

/쿠링 패키지 빌드

Copy link

🔨 iOS 17.0.1 iPhone 15 Pro 에서 스위프트 패키지를 빌드합니다.

Copy link

✅ 패키지를 성공적으로 빌드했습니다.

lgvv
lgvv previously approved these changes Sep 10, 2024
@lgvv
Copy link
Member

lgvv commented Sep 11, 2024

/쿠링 머지

@github-actions github-actions bot added the automerge auto merge 를 수동으로 미리 활성화 하는 경우 라벨을 추가해줘야 합니다. "/쿠링 머지" 실행시 자동으로 추가. label Sep 11, 2024
Copy link

🗿 머지 시도 실패. 승인 개수, 테스트 성공 여부 등 자동 머지 조건을 만족하는지 확인하세요.

@sunshiningsoo
Copy link
Member Author

/쿠링 패키지 빌드

Copy link

🔨 iOS 17.0.1 iPhone 15 Pro 에서 스위프트 패키지를 빌드합니다.

@lgvv
Copy link
Member

lgvv commented Sep 11, 2024

/쿠링 머지

Copy link

🗿 머지 시도 실패. 승인 개수, 테스트 성공 여부 등 자동 머지 조건을 만족하는지 확인하세요.

Copy link

✅ 패키지를 성공적으로 빌드했습니다.

@sunshiningsoo
Copy link
Member Author

/쿠링 머지

@github-actions github-actions bot merged commit 42d6065 into develop/2.0.0 Sep 11, 2024
Copy link

🕊️ PR #233이 머지되었습니다.

@github-actions github-actions bot deleted the enhance/sungsoo/searchHistory branch September 11, 2024 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge auto merge 를 수동으로 미리 활성화 하는 경우 라벨을 추가해줘야 합니다. "/쿠링 머지" 실행시 자동으로 추가. 새기능/개선작업 새 기능 작업 또는 개선 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants