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

[Exiled::Core] Assets and Fragments #2568

Merged
merged 32 commits into from
Jul 1, 2024
Merged

[Exiled::Core] Assets and Fragments #2568

merged 32 commits into from
Jul 1, 2024

Conversation

NaoUnderscore
Copy link
Collaborator

  • Added new classes and attributes for improved entity management.
  • Added AssetFragment, EClass, EntityAsset, and related attributes.
  • Any types implementing IAssetFragment can be used to generate a fragment or an EntityAsset.
  • Began adding EProperty attributes to classes which are supposed to be used to generate fragments and assets.

- Added new classes and attributes for improved entity management.
- Added AssetFragment, EClass, EntityAsset, and related attributes.
- Any types implementing IAssetFragment can be used to generate a fragment or an EntityAsset.
- Began adding EProperty attributes to classes which are supposed to be used to generate fragments and assets.
@NaoUnderscore NaoUnderscore added enhancement New feature or request regarding-api An issue or PR targeting the Exiled API project requires-testing Things need to be verified by an Exiled Developer/Contributor P1 First Priority labels May 20, 2024
@NaoUnderscore NaoUnderscore added this to the Exiled 9 milestone May 20, 2024
@NaoUnderscore NaoUnderscore self-assigned this May 20, 2024
Copy link
Contributor

@VALERA771 VALERA771 left a comment

Choose a reason for hiding this comment

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

Already looks pretty good but little complicated for basic users. I agree that we will need a guide on how to use it.

@iamalexrouse
Copy link
Member

@NaoUnderscore How test, need examples on scenarios.

@NaoUnderscore NaoUnderscore marked this pull request as ready for review May 24, 2024 11:44
# Conflicts:
#	Exiled.API/Features/CustomStats/CustomHealthStat.cs
@iamalexrouse
Copy link
Member

@Misfiy Hey. Have you done a test and confirmed the PR doesn't produce leaks?

Thanks,
Alex

@Misfiy
Copy link
Contributor

Misfiy commented May 31, 2024

Untested. I have only fixed conflicts.

@Misfiy Misfiy added the Do not merge Don't merge this PR label Jun 22, 2024
@Misfiy
Copy link
Contributor

Misfiy commented Jun 22, 2024

NOTE: It is do not merge because of possible lag caused by it

@Misfiy Misfiy marked this pull request as draft June 24, 2024 11:01
@Misfiy Misfiy marked this pull request as ready for review June 25, 2024 06:26
@Misfiy Misfiy removed the Do not merge Don't merge this PR label Jun 25, 2024
@iamalexrouse
Copy link
Member

Testing now...

@Misfiy Misfiy added tested This PR has been built and tested and does not cause any obvious errors. documentation Improvements or additions to documentation and removed requires-testing Things need to be verified by an Exiled Developer/Contributor labels Jun 30, 2024
@iamalexrouse
Copy link
Member

@Misfiy @NaoUnderscore

I haven't found any other issues during testing except like 1 null which was fixed, everything should be fine.

@Misfiy Misfiy added the bug Something isn't working label Jun 30, 2024
@NaoUnderscore NaoUnderscore merged commit 1b8d3a5 into apis-rework Jul 1, 2024
4 checks passed
NaoUnderscore added a commit that referenced this pull request Jul 1, 2024
* `[Exiled::Core]` `Assets` and `Fragments` (#2568)

* Core's assets have been implemented:

- Added new classes and attributes for improved entity management.
- Added AssetFragment, EClass, EntityAsset, and related attributes.
- Any types implementing IAssetFragment can be used to generate a fragment or an EntityAsset.
- Began adding EProperty attributes to classes which are supposed to be used to generate fragments and assets.

* Implemented `EProperty` for Items

* ?.

* Minor changes to `EConfig`. Added an `EClass`.
Converted `Effect` class to struct.

* Added `Generator` `EClass`

* Fix hint's `EClass`

* Added more `EClass`'es

* Fixed `AssetFragment::GenerateAsset` generating assets from types which are not marked as `EClass`.

* Fixed `State` compare by ref instead of value

* Minor fixes

* All `EClass`'es

* Fix warnings

* Fix docs

* Revert CustomHealthStat.cs

* Fix build error

* fix conflict?

* fix typo

* fix removals

* fix lag & log spam

* Changes & null fix

* [Core API] Fix null & invalid cast (#2650)

---------

Co-authored-by: Nameless <85962933+Misfiy@users.noreply.github.com>

* Revert "Merge remote-tracking branch 'origin/dev' into apis-rework"

This reverts commit 08e279f, reversing
changes made to 1b8d3a5.

---------

Co-authored-by: Nameless <85962933+Misfiy@users.noreply.github.com>
NaoUnderscore added a commit that referenced this pull request Jul 1, 2024
* `[Exiled::Core]` `Assets` and `Fragments` (#2568)

* Core's assets have been implemented:

- Added new classes and attributes for improved entity management.
- Added AssetFragment, EClass, EntityAsset, and related attributes.
- Any types implementing IAssetFragment can be used to generate a fragment or an EntityAsset.
- Began adding EProperty attributes to classes which are supposed to be used to generate fragments and assets.

* Implemented `EProperty` for Items

* ?.

* Minor changes to `EConfig`. Added an `EClass`.
Converted `Effect` class to struct.

* Added `Generator` `EClass`

* Fix hint's `EClass`

* Added more `EClass`'es

* Fixed `AssetFragment::GenerateAsset` generating assets from types which are not marked as `EClass`.

* Fixed `State` compare by ref instead of value

* Minor fixes

* All `EClass`'es

* Fix warnings

* Fix docs

* Revert CustomHealthStat.cs

* Fix build error

* fix conflict?

* fix typo

* fix removals

* fix lag & log spam

* Changes & null fix

* [Core API] Fix null & invalid cast (#2650)

---------

Co-authored-by: Nameless <85962933+Misfiy@users.noreply.github.com>

* Revert "Merge remote-tracking branch 'origin/dev' into apis-rework"

This reverts commit 08e279f, reversing
changes made to 1b8d3a5.

* Exiled9  (#2641)

* Changes

* that

---------

Co-authored-by: Yamato <66829532+louis1706@users.noreply.github.com>

---------

Co-authored-by: Nameless <85962933+Misfiy@users.noreply.github.com>
Co-authored-by: Yamato <66829532+louis1706@users.noreply.github.com>
NaoUnderscore added a commit that referenced this pull request Jul 1, 2024
* `[Exiled::Core]` `Assets` and `Fragments` (#2568)

* Core's assets have been implemented:

- Added new classes and attributes for improved entity management.
- Added AssetFragment, EClass, EntityAsset, and related attributes.
- Any types implementing IAssetFragment can be used to generate a fragment or an EntityAsset.
- Began adding EProperty attributes to classes which are supposed to be used to generate fragments and assets.

* Implemented `EProperty` for Items

* ?.

* Minor changes to `EConfig`. Added an `EClass`.
Converted `Effect` class to struct.

* Added `Generator` `EClass`

* Fix hint's `EClass`

* Added more `EClass`'es

* Fixed `AssetFragment::GenerateAsset` generating assets from types which are not marked as `EClass`.

* Fixed `State` compare by ref instead of value

* Minor fixes

* All `EClass`'es

* Fix warnings

* Fix docs

* Revert CustomHealthStat.cs

* Fix build error

* fix conflict?

* fix typo

* fix removals

* fix lag & log spam

* Changes & null fix

* [Core API] Fix null & invalid cast (#2650)

---------

Co-authored-by: Nameless <85962933+Misfiy@users.noreply.github.com>

* Revert "Merge remote-tracking branch 'origin/dev' into apis-rework"

This reverts commit 08e279f, reversing
changes made to 1b8d3a5.

* Exiled9  (#2641)

* Changes

* that

---------

Co-authored-by: Yamato <66829532+louis1706@users.noreply.github.com>

* Moved modified code. (#2656)

---------

Co-authored-by: Nameless <85962933+Misfiy@users.noreply.github.com>
Co-authored-by: Yamato <66829532+louis1706@users.noreply.github.com>
Co-authored-by: Alex Rouse <123724383+ALEXWARELLC@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request P1 First Priority regarding-api An issue or PR targeting the Exiled API project tested This PR has been built and tested and does not cause any obvious errors.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants