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

Fix issue with pillar ability's cover not updating properly on units spawned next to the actor #1293

Closed
wants to merge 3 commits into from

Conversation

BlackDog86
Copy link
Contributor

Fixes #1288

@BlackDog86 BlackDog86 self-assigned this Dec 19, 2023
@BlackDog86 BlackDog86 added bug-basegame ready-to-review A pull request is ready to be reviewed labels Dec 19, 2023
@BlackDog86 BlackDog86 added this to the 1.28.0 milestone Dec 19, 2023
@BlackDog86 BlackDog86 changed the title Added code by Sigma 1234 which was used in MZ perk pack to fix pillar destructible cover spawning Fix issue with pillar ability's cover not updating properly on units spawned next to the actor Dec 19, 2023
Copy link
Contributor

@Iridar Iridar left a comment

Choose a reason for hiding this comment

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

This PR is missing .x2proj change for X2Effect_Pillar.

@BlackDog86
Copy link
Contributor Author

Should be done now

@Iridar
Copy link
Contributor

Iridar commented Feb 27, 2024

Sorry, but it's in the wrong commit now. For PRs that add new files to Highlander, the first commit must have the new file and change to .x2proj file (it helps to restart modbuddy before making the commit so that changes actually apply to the .x2proj file), and the second commit should have the rest of the PR.

@BlackDog86
Copy link
Contributor Author

Commits re-arranged

@BlackDog86 BlackDog86 added waiting-on-author A pull request is waiting on changes from the author and removed ready-to-review A pull request is ready to be reviewed labels Feb 27, 2024
@BlackDog86
Copy link
Contributor Author

Needs more work - issue was more complex than first thought

@Iridar
Copy link
Contributor

Iridar commented Feb 27, 2024

In testing this PR doesn't actually fix the issue. The latest commit includes other things I tried, but none of them help either. I'm closing it for now, feel free to reopen later if a solution is found.

@Iridar Iridar closed this Feb 27, 2024
@Iridar Iridar removed this from the 1.28.0 milestone May 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug-basegame waiting-on-author A pull request is waiting on changes from the author
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Templars' Pillar doesn't give cover when spawned directly adjacent to units
2 participants