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

Vehicles Support, PlotSquare API update and Material FIX #31

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

pollitoyeye
Copy link

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).

@@ -172,7 +172,7 @@ void setName(Player p, ArmorStand as) {
p.sendMessage(ChatColor.RED + Config.noAirError);
return;
}
b.setType(Material.SIGN);
b.setType(Material.OAK_WALL_SIGN);

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
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.

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.

Copy link
Author

@pollitoyeye pollitoyeye May 10, 2020

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

@@ -184,7 +184,7 @@ void setPlayerSkull(Player p, ArmorStand as) {
p.sendMessage(ChatColor.RED + Config.noAirError);
return;
}
b.setType(Material.SIGN);
b.setType(Material.OAK_WALL_SIGN);

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
Author

Choose a reason for hiding this comment

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

@Andre601
Copy link

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

@pollitoyeye
Copy link
Author

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

@Andre601
Copy link

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
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

@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