Skip to content

Vehicles Support, PlotSquare API update and Material FIX#31

Open
pollitoyeye wants to merge 4 commits into
St3venAU:masterfrom
pollitoyeye:master
Open

Vehicles Support, PlotSquare API update and Material FIX#31
pollitoyeye wants to merge 4 commits into
St3venAU:masterfrom
pollitoyeye:master

Conversation

@pollitoyeye

Copy link
Copy Markdown

St3venAU, please contact me on Spigot via DM after accepting this so I can give you a free copy of the Vehicles plugin (So you can compile the plugin).

return;
}
b.setType(Material.SIGN);
b.setType(Material.OAK_WALL_SIGN);

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Shouldn't this be a Sign post?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

SIGN_POST isn't a Material in 1.15 API.
Maybe the Material would be OAK_SIGN.
I'm not sure how that part is supposed to work.

However if the plugin is intended to be compiled in 1.13 just remove that part before merging.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

From what I understand is the method(s) used with this used for things like renaming the armor stand or adding a Player head.
In that case does AST "summon" a Sign which is "opened" for the player to then input the text.
And it was always a standing sign. I'm not sure if a wall sign would change this dramatically, but I would leave it as a standing sign for consistency.

@pollitoyeye pollitoyeye May 10, 2020

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I checked and it works the same. However I just changed it from OAK_WALL_SIGN to OAK_SIGN

return;
}
b.setType(Material.SIGN);
b.setType(Material.OAK_WALL_SIGN);

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Shouldn't this be a Sign post?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

@Andre601

Copy link
Copy Markdown

@pollitoyeye Wouldn't it be better to have a downloadable, public API, rather than requiring to have the Jar available?

@pollitoyeye

Copy link
Copy Markdown
Author

Do you know any program that generates an API Jar?
Doing it myself for every change is a pain.

@Andre601

Copy link
Copy Markdown

there exists jitpack.io which would allow you to share a GitHub repository like an API for maven/gradle.
Just post the code for your API in a repo, publish it, create a release and give the instructions on how to integrate the API into maven/gradle and it should be good to go.
And most if not all distribution centres like Bintray, for example, allow you to publish and share APIs for free. (I use it)

@pollitoyeye

Copy link
Copy Markdown
Author

Okay, I found a project called JstubGenerator to do this.
I uploaded the API jar to the plugin page.
This is the download link:
https://www.mediafire.com/file/m15tceksbeipa36/Vehicles%2810.6%29-API.jar/file

@Andre601

Copy link
Copy Markdown

@St3venAU Will this ever be merged/looked into?
I'm pretty sure people want this to (finally) get added as some use PlotSquared but can't use the latest version because AST doesn't support their new API yet.
Either that or they would need to remove AST from their server and therefore lose some functionalities.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants