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

Port to 1.21 NeoForge #93

Open
Karotte128 opened this issue Aug 7, 2024 · 11 comments
Open

Port to 1.21 NeoForge #93

Karotte128 opened this issue Aug 7, 2024 · 11 comments
Milestone

Comments

@Karotte128
Copy link

Describe your idea (If you have more than one idea, please open the rest in other issues)

Please port this mod to 1.21 Neoforge

Describe alternatives you've considered if any

Additional context

Linked Issues

No response

@zyxkad
Copy link
Collaborator

zyxkad commented Aug 7, 2024

1.21 just released, as well as CC:T don't have a stable 1.21 version too.

@zyxkad zyxkad added the upstream Depends on upstream's update label Aug 12, 2024
@jamieb999
Copy link

#98

This needs to be updated to at least 21.1 (1.21.1) I mean it is stable. I opened a ticket about this
and they closed it as not saying that it is not being planned. Wow Just wow they won't even
keep the mod updated.

Uninstall this mod if the creator won't even keep the mod updated. I am done with
it!

@SirEndii
Copy link
Member

SirEndii commented Nov 11, 2024

I am not someone who has 60 hours a week time to work on every project. I have a job, a real life and can't spend every hour on AP
And if you would've looked on the commit history, you would see that I currently put a lot of work into AP but the port needs a bit of time

I only closed your issue because it was a duplicate and rude

@SirEndii SirEndii added this to the 0.7.x milestone Nov 11, 2024
@zyxkad zyxkad changed the title 1.21 Port to 1.21 NeoForge Nov 11, 2024
@jamieb999
Copy link

jamieb999 commented Nov 11, 2024

I wasn't rude at all I just made a feature request that was it not rude in any way.
it is also impossible to tell if some one is "rude" from a text message like that
you are reading into something that isn't there at all.

I was in no way rude at all. Also for the record the ticket was not closed as a duplicate
which it should have been then it was closed off as not being planned to be done
or something like that. The ticket should have been closed off as "duplicate" and
put in your comment duplicate of ticket and the ticket number and the link that is
standard procedure here on Github and any other development places like this.

It definately needs to be ported to 1.21.1 (21.1) at least please for Neoforged
as that is the stable version.

@zyxkad
Copy link
Collaborator

zyxkad commented Nov 11, 2024

We not only want stable version for Neoforged, but also require CC:T publish a stable API, otherwise people will just asking for mod incompatible error forever.

As well as, not planned include duplicate
Screenshot 2024-11-10 at 18 18 00

@jamieb999
Copy link

1.21.1 IS a stable version 1.21.2 is UNSTABLE So again it is stable Please update for Neoforged 1.21.1 (21.1) please

Not planned dosn't include duplicate what so ever again you are making this stuff up as
you go and refuse to admit any wrong doing here.

Please update to Neoforged 1.21.1 (21.1) please.

@zyxkad
Copy link
Collaborator

zyxkad commented Nov 11, 2024

I agree it will be better to update things. However currently we don't have time to update this, maybe you can make a PR for this update please?

PS I have no idea why Github says Not planned included duplicate, but if you are using it you better learn this definition

@zyxkad
Copy link
Collaborator

zyxkad commented Nov 11, 2024

Also, I just checked and there is no stable version for 1.21.1 CC:T, would you mind paste the link of where you get that?

@jamieb999
Copy link

There is a version if CC:Tweaked for 1.21.1

https://modrinth.com/mod/cc-tweaked/version/CRVodEhR

@zyxkad
Copy link
Collaborator

zyxkad commented Nov 11, 2024

Screenshot 2024-11-10 at 19 23 03

Maybe you understand what does alpha mean
In case you don't understand, I can tell you that means this version is not stable, and is not well tested.

@SirEndii
Copy link
Member

wasn't rude at all I just made a feature request that was it not rude in any way.

It is just annoying for a developer to read feature requests like "you NEED to xyz".
Sorry that I forgot to mark it as a duplicate, it is just really frustrating
Issues like that will not speed up the process

The only thing I "need" to do is to find the time to do this, this isn't just "change a number and it works". If I have the hours of time to port it, I can do it. But I don't have that currently
And if another dev sees this issue and thinks, "hey, I could help AP a bit and try to start a port with a PR", that would be really appreciated

So please try to respect the time of developers and their personal life in the future more, that's a thing that you need to do

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

No branches or pull requests

4 participants