plugin-hub icon indicating copy to clipboard operation
plugin-hub copied to clipboard

harry-potter-spells

Open JonasDJ1999 opened this issue 1 year ago • 23 comments

A plugin that makes you say spells from Harry Potter, when casting certain spells.

JonasDJ1999 avatar May 15 '24 12:05 JonasDJ1999

New plugin harry-potter-spells: https://github.com/JonasDJ1999/Harry-Potter-Spells/tree/af2cf0e8bea7697be99b3310370a2dc821c312ef

Do not close and open new PRs. Push to the same branch.

YvesW avatar May 15 '24 12:05 YvesW

You've added 2 plugin files "harry-potter-spells" and "/plugins/Harry Potter Spells". There should only be 1 file added and it should be in the "plugins" directory.

Delete the "harry-potter-spells" file you've created and rename the "Harry Potter Spells" file in the "plugins" directory to "harry-potter-spells" because afaik you can't have spaces in the plugin file name.

DapperMickie avatar May 15 '24 12:05 DapperMickie

You also might want to change this: https://github.com/JonasDJ1999/Harry-Potter-Spells/blob/Harry-Potter-Spells/build.gradle#L26 to "com.harrypotterspells".

DapperMickie avatar May 15 '24 12:05 DapperMickie

sorry im very noob....

JonasDJ1999 avatar May 15 '24 12:05 JonasDJ1999

You've added 2 plugin files "harry-potter-spells" and "/plugins/Harry Potter Spells". There should only be 1 file added and it should be in the "plugins" directory.

Delete the "harry-potter-spells" file you've created and rename the "Harry Potter Spells" file in the "plugins" directory to "harry-potter-spells" because afaik you can't have spaces in the plugin file name.

how do I delete that one specifically?

JonasDJ1999 avatar May 15 '24 12:05 JonasDJ1999

You've added 2 plugin files "harry-potter-spells" and "/plugins/Harry Potter Spells". There should only be 1 file added and it should be in the "plugins" directory. Delete the "harry-potter-spells" file you've created and rename the "Harry Potter Spells" file in the "plugins" directory to "harry-potter-spells" because afaik you can't have spaces in the plugin file name.

how do I delete that one specifically?

If you're using the github website to do it, just navigate to the file and click the button with 3 dots on the top right and click delete. https://gyazo.com/e099a9b487449454e69d14ede0ebdd10

If you've got the repo cloned to your pc, just delete the file and make a commit.

DapperMickie avatar May 15 '24 12:05 DapperMickie

Here is a link to the file that should be deleted: https://github.com/JonasDJ1999/plugin-hub/blob/master/harry-potter-spells

DapperMickie avatar May 15 '24 12:05 DapperMickie

Here is a link to the file that should be deleted: https://github.com/JonasDJ1999/plugin-hub/blob/master/harry-potter-spells

okay I deleted that and renamed the other, what now?

JonasDJ1999 avatar May 15 '24 12:05 JonasDJ1999

Here is a link to the file that should be deleted: https://github.com/JonasDJ1999/plugin-hub/blob/master/harry-potter-spells

okay I deleted that and renamed the other, what now?

Commit the changes :)

DapperMickie avatar May 15 '24 12:05 DapperMickie

Here is a link to the file that should be deleted: https://github.com/JonasDJ1999/plugin-hub/blob/master/harry-potter-spells

okay I deleted that and renamed the other, what now?

Commit the changes :)

how do I do that without deleting this entire thread lol?

JonasDJ1999 avatar May 15 '24 12:05 JonasDJ1999

If you did it through the github webpage, there is a button 'commit changes' that appears where the dots were after you press delete.
If you did it locally on a clone:

git commit
git push

Fiskmans avatar May 15 '24 12:05 Fiskmans

Spells in Harry Potter are copyrighted material and they have been very diligent in defending their copyright over the years. Thus I don't think this should be added.

raiyni avatar May 15 '24 12:05 raiyni

Spells in Harry Potter are copyrighted material and they have been very diligent in defending their copyright over the years. Thus I don't think this should be added.

can just rename them then if that's needed ^^

JonasDJ1999 avatar May 15 '24 12:05 JonasDJ1999

Spells in Harry Potter are copyrighted material and they have been very diligent in defending their copyright over the years. Thus I don't think this should be added.

yes I was unsure about what extend this was to. to fix this issue Ive just made the spells goofy/memes with a config where you can enter the text you want. hopefully I can later incorporate a way have a list of suggestions. so this is taken care off

JonasDJ1999 avatar May 15 '24 18:05 JonasDJ1999

if anyone is curious about the future for this plugin. hopefully I can learn more about plugins and coding and get the following features: -a picture for the plugin (Kodai Wand maybe hmm). -cure, heal and ancients spells (couldn't find their casting ID) -% toggle to be by itself in a different panel

JonasDJ1999 avatar May 15 '24 18:05 JonasDJ1999

Spells in Harry Potter are copyrighted material and they have been very diligent in defending their copyright over the years. Thus I don't think this should be added.

can just rename them then if that's needed ^^

I suspect the name of the plugin itself may be an issue.

cdfisher avatar May 15 '24 18:05 cdfisher

Spells in Harry Potter are copyrighted material and they have been very diligent in defending their copyright over the years. Thus I don't think this should be added.

can just rename them then if that's needed ^^

I suspect the name of the plugin itself may be an issue.

then I can change it to "Custom Spells". but ya, I will wait and if this gets approved or not first, then Im told to change the name, I must figure out how to do that without ruining everything lmfao

JonasDJ1999 avatar May 15 '24 18:05 JonasDJ1999

Spells in Harry Potter are copyrighted material and they have been very diligent in defending their copyright over the years. Thus I don't think this should be added.

can just rename them then if that's needed ^^

I suspect the name of the plugin itself may be an issue.

then I can change it to "Custom Spells". but ya, I will wait and if this gets approved or not first, then Im told to change the name, I must figure out how to do that without ruining everything lmfao

You can't change the internal name once it's on the hub.

cdfisher avatar May 15 '24 18:05 cdfisher

Spells in Harry Potter are copyrighted material and they have been very diligent in defending their copyright over the years. Thus I don't think this should be added.

can just rename them then if that's needed ^^

I suspect the name of the plugin itself may be an issue.

then I can change it to "Custom Spells". but ya, I will wait and if this gets approved or not first, then Im told to change the name, I must figure out how to do that without ruining everything lmfao

You can't change the internal name once it's on the hub.

ouff what would I have to do then? make a whole new one and copy the code in? lmao

JonasDJ1999 avatar May 15 '24 18:05 JonasDJ1999

tf Im pretty sure I wrote it, but I changed the coding, so you input what spells you want yourself so no copyright

JonasDJ1999 avatar May 30 '24 20:05 JonasDJ1999

its been a month now lol. if there is any problems, please tell me so I can fix them

JonasDJ1999 avatar Jun 15 '24 16:06 JonasDJ1999

As cdfisher mentioned it's unlikely we would want to accept a plugin explicitly including Harry Potter in it's name, so you'll need to change that.

The name of your actual Github repo is not related to the name of the plugin, but if you wanted to do that it can be done in the repo settings. The changes we would actually want is the removal of the references from the actual code (e.g. setting names/descriptions) as well as this PR should be replacing your current change of creating the file harry-potter-spells and instead creating a file of whatever new name you choose and linking to the new repo name if changed.

Felanbird avatar Jun 15 '24 16:06 Felanbird

Marking as draft for now. Please make the requested changes, then mark the plugin as ready for review.

LlemonDuck avatar Jul 09 '24 17:07 LlemonDuck

@Felanbird sorry for late reply, Ive been on vacation. How do I change the name, like where? and without messing everything up?

JonasDJ1999 avatar Jul 31 '24 17:07 JonasDJ1999

Gonna close this as very stale. Feel free to resubmit once you've made the requested changes.

How do I change the name, like where? and without messing everything up?

The plugin is not yet released to anyone, so you're free to edit it as much as you want. Nothing will be messed up on our end.

LlemonDuck avatar Oct 15 '24 21:10 LlemonDuck