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

Add support for NC 30 #216

Merged
merged 1 commit into from
Dec 13, 2024
Merged

Add support for NC 30 #216

merged 1 commit into from
Dec 13, 2024

Conversation

IndrekHaav
Copy link
Contributor

@IndrekHaav IndrekHaav commented Dec 6, 2024

The app seems to work fine, this just marks it as compatible.
resolves #200

The app seems to work fine, this just marks it as compatible.

Signed-off-by: Indrek Haav <IndrekHaav@users.noreply.github.com>
@IndrekHaav
Copy link
Contributor Author

Looks like @datenangebot hasn't been active for a while. Pinging @nickvergessen, not sure who else can merge PRs...

@kenda kenda merged commit 11ff84f into nextcloud:main Dec 13, 2024
11 of 14 checks passed
@kenda
Copy link
Collaborator

kenda commented Dec 13, 2024

Merged your PR but I think we would need a version bump + release to get this out. But I'm not familiar anymore how the release process worked in the last time here.

@nickvergessen
Copy link
Member

But I'm not familiar anymore how the release process worked in the last time here.

If you have the app key, it should be something like:

@kenda
Copy link
Collaborator

kenda commented Dec 13, 2024

If you have the app key, it should be something like:

No, I don't have the app key. The releases were always done by @datenangebot in the past afaik.

@IndrekHaav IndrekHaav deleted the nc30 branch December 13, 2024 21:14
@IndrekHaav
Copy link
Contributor Author

Does this mean the app is effectively unmaintained? Considering the only person who can make a new release appears to be inactive.

@nickvergessen
Copy link
Member

@datenangebot
Copy link
Collaborator

Hi, well yes, the app is without a real maintainer at the moment.
And there are known security problems in this app, that's why I don't feel good to ship new releases.
Unfortunately I can't find any time to fix that...

If I can support you to get a new sign key or if there is a handover process, let me know.

@IndrekHaav
Copy link
Contributor Author

If I can support you to get a new sign key or if there is a handover process, let me know.

@datenangebot If you meant me, then unfortunately I have neither the time nor experience to become a NC app maintainer.

If a new version with NC 30 support is not happening, then perhaps there should be an announcement about this. As you can see from the linked issue, there are quite a few people waiting for an update.

@solonovamax
Copy link

And there are known security problems in this app, that's why I don't feel good to ship new releases.

like what? or can you not disclose them publicly?

because if they're able to be disclosed, then it would be possible for someone else to go about fixing them

@BraidenPsiuk
Copy link

If that information can't easily be disclosed, maybe this can be answered- Is it worth spending time fixing these security problems? Or is it better to spend the time developing a new app from scratch that follows more modern practices?

And if a new app is developed, is there a good strategy for "sunsetting" old apps and replacing them with new ones?

As a user of this app for a few years now I have unfortunately felt the lack of updates and care for this app, it is almost always the last to get support when upgrading to new NC versions.

@IndrekHaav IndrekHaav mentioned this pull request Dec 30, 2024
@datenangebot
Copy link
Collaborator

And if a new app is developed, is there a good strategy for "sunsetting" old apps and replacing them with new ones?

If there is somebody who wants to renew this app by reusing the name and technicaly the namespace, please contact me.

My 50cents: Do we really need this app renewed? I think tracking data can be done by the tables application and generating diagramms and charts can be archieved with the combination of tables and analytics... An health app should be more specific to health topics, allowing specific calculations and so on, but that takes a lot of time and needs expert knowledge. I tried several times to get a team around this topic within the nextcloud company, some community guys and the GNU health organisation, but unfortunately without luck. 🤔

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.

Nextcloud 30 support
6 participants