Skip to content

86 boss skill silence #95

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
Open
nonoaa wants to merge 4 commits into
base: main
Choose a base branch
from
Open

86 boss skill silence #95

nonoaa wants to merge 4 commits into from

Conversation

@nonoaa
Copy link
Collaborator

@nonoaa nonoaa commented Nov 21, 2022

작업내용

  • 사일런스 스킬 버그 수정
    • 중복된 타워를 타겟팅하거나 남은 타워가 있음에도 타겟팅하지 않는 버그를 수정했습니다.
      • C# 에서 새로운 List를 생성하고 =으로 복사하면 복사한 List의 원소를 Remove해도 원본의 List원소에는 영향이 없을줄 알았는데 원본의 List의 원소도 같이 Remove되네요.
      • 새로운 List를 만들고 원본의 값들을 for문으로 돌며 Add 하는 방법으로 수정했습니다.
  • 스킬 이펙트 추가
    • 스킬 이펙트들을 추가했습니다.
    • image

@nonoaa nonoaa linked an issue Nov 21, 2022 that may be closed by this pull request
Boss Skill: Silence #86
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@giftof giftof giftof approved these changes

@yugd1211 yugd1211 yugd1211 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Boss Skill: Silence

4 participants

@nonoaa @giftof @yugd1211