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

Added AdvancedEnchantments #73

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Added AdvancedEnchantments #73

wants to merge 1 commit into from

Conversation

WillFP
Copy link

@WillFP WillFP commented Nov 8, 2021

Added AdvancedEnchantments - the list of problems is so big that it would be longer than the entire rest of the readme

@KaspianDev
Copy link
Contributor

Also mention that his website shows prices higher than even possible on spigot (to make discounts look even bigger)

also you cant review plugins on his site, or dislike other reviews (you can only like them, dislike button is not functioning)

@WillFP
Copy link
Author

WillFP commented Jan 27, 2022

update: complaining in the discussion tab in competitors to AE, and has been for over 2 years now:
feb 2020:
Screenshot 2022-01-27 at 14 36 09

jan 2022:
Screenshot 2022-01-27 at 14 36 34

Link: https://www.spigotmc.org/threads/eliteenchantments-%E2%9E%9C-create-your-own-enchantments-%E2%9A%9C%EF%B8%8F-web-panel-%E2%9A%9C%EF%B8%8F-190-premade-enchants-27-gkits.417799/page-10

this guy really is the bottom of the barrel

@KaspianDev
Copy link
Contributor

change it from AE to GC. in general

@WillFP
Copy link
Author

WillFP commented Jan 27, 2022

oh - and about the roman numeral code? yeah here's the stack overflow answer where he pasted the entire thing from - main method, class name, and all https://stackoverflow.com/a/9073310/11427550

@Wesley51
Copy link
Contributor

Wesley51 commented Jul 22, 2022

Wierd that a competitor would make this PR 🤔

@WillFP
Copy link
Author

WillFP commented Aug 25, 2022

Competitor that doesn't have a 3000-line long switch statement

@Wesley51
Copy link
Contributor

oh - and about the roman numeral code? yeah here's the stack overflow answer where he pasted the entire thing from - main method, class name, and all https://stackoverflow.com/a/9073310/11427550

So what? All the class does is convert numbers... it doesn't contribute to the core functionality of the plugin. It would be a different story if he was reselling other plugin's code, which afaik is not the case. I don't see an issue here.

To me, this looks like you are using this public list to get more sales by getting people to believe your plugin is better. It's not right.

@KaspianDev
Copy link
Contributor

KaspianDev commented Aug 25, 2022

oh - and about the roman numeral code? yeah here's the stack overflow answer where he pasted the entire thing from - main method, class name, and all https://stackoverflow.com/a/9073310/11427550

So what? All the class does is convert numbers... it doesn't contribute to the core functionality of the plugin. It would be a different story if he was reselling other plugin's code, which afaik is not the case. I don't see an issue here.

To me, this looks like you are using this public list to get more sales by getting people to believe your plugin is better. It's not right.

except his plugin is libre software

@WillFP
Copy link
Author

WillFP commented Aug 30, 2022

oh - and about the roman numeral code? yeah here's the stack overflow answer where he pasted the entire thing from - main method, class name, and all https://stackoverflow.com/a/9073310/11427550

So what? All the class does is convert numbers... it doesn't contribute to the core functionality of the plugin. It would be a different story if he was reselling other plugin's code, which afaik is not the case. I don't see an issue here.

To me, this looks like you are using this public list to get more sales by getting people to believe your plugin is better. It's not right.

https://paste.willfp.com/hepeminaro.java the main class that controls all the effects is a gigantic switch statement

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.

3 participants