Skip to content

JSON Support for plugins#28

Closed
educatalan02 wants to merge 2 commits into
RocketModFix:devfrom
RedstonePlugins:master
Closed

JSON Support for plugins#28
educatalan02 wants to merge 2 commits into
RocketModFix:devfrom
RedstonePlugins:master

Conversation

@educatalan02
Copy link
Copy Markdown
Collaborator

UseJsonForPlugins bool at Rocket.Config.xml

Now you can add comments to the JSON config file
Code:
[JsonConverter(typeof(JsonExtension), "Name")] public string Name = "test";
Result:
{ "Name": "test"/*Name*/, }

@educatalan02 educatalan02 added the enhancement New feature or request label Oct 31, 2023
@mkspeed0
Copy link
Copy Markdown
Collaborator

Hi! The idea is very good and was considered by me the other day. First of all, you should open an Issue with this topic and find out the opinion of others. Then publish a Pull Request. So another erroneous point, do not close your pull requests as well as issues. Sunnamed434 closes the pull request itself, doing so by publishing.

@educatalan02
Copy link
Copy Markdown
Collaborator Author

Purpose of this is to discuss It here, no matter if its an issue or not. I closed the pr because I forgot to removed unused Code.

@educatalan02 educatalan02 changed the base branch from master to dev October 31, 2023 17:43
Copy link
Copy Markdown
Member

@sunnamed434 sunnamed434 left a comment

Choose a reason for hiding this comment

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

Looks good to me

@sunnamed434
Copy link
Copy Markdown
Member

Please make another PR or bump the latest updates from the master branch of this repo, because this one has conflicts due to an outdated branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants