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

Update drinks_cups.yml #570

Merged
merged 10 commits into from
Nov 10, 2024
Merged

Conversation

YmepIIIij-TeRorIst
Copy link
Contributor

@YmepIIIij-TeRorIst YmepIIIij-TeRorIst commented Nov 9, 2024

Просто заменил parent и всё корректно заработало.
Везде кружки CP14 работают теперь. А я парился над этим 2-3 часа.

fix #563 @TheShuEd

@github-actions github-actions bot added the No C# label Nov 9, 2024
Copy link
Contributor

@TheShuEd TheShuEd left a comment

Choose a reason for hiding this comment

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

Не трогать ванильные объекты. Это изменение добавило в билд все ванильные кружки + добавило им весь функционал склянок, включая разбивание на стекло.

@@ -1,7 +1,7 @@
#Base for future bottles
- type: entity
name: Green Bottle
parent: CP14BaseVial
parent: [CP14BaseVial, CP14BaseVialFragile]
Copy link
Contributor

Choose a reason for hiding this comment

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

а зачем, если CP14BaseVialFragile и так наследуется от CP14BaseVial?

image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

А там разбиваемое стекло

Copy link
Contributor

Choose a reason for hiding this comment

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

и?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

разницы короче нету, что так и что так. Я просто много не думал

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Хоть поставить без CP14BaseVial

Copy link
Contributor

Choose a reason for hiding this comment

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

думать полезно

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Я знаю, но я подал так с другой точки зрения. хотя и так и так будет работать, можно сказать я не внимательный.

@TheShuEd
Copy link
Contributor

благодарим-с

@TheShuEd TheShuEd merged commit 45dc473 into crystallpunk-14:master Nov 10, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Inconsistency of fluid containers
2 participants