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

tt-rss: Fix bug in wizard scripts #5955

Merged
merged 5 commits into from
Dec 9, 2023

Conversation

mreid-tt
Copy link
Contributor

@mreid-tt mreid-tt commented Dec 9, 2023

Description

This addresses a bug identified in pull request #5923, where the installation wizard incorrectly detects multiple PHP profiles. This issue is specific to DSM 7 installations, making the concern irrelevant for these cases. It's important to clarify that the initial implementation was intended as an enhancement exclusively for DSM 6 users.

Fixes #

Checklist

  • Build rule all-supported completed successfully
  • New installation of package completed successfully
  • Package upgrade completed successfully (Manually install the package again)
  • Package functionality was tested
  • Any needed documentation is updated/created

Type of change

  • Bug fix

@mreid-tt mreid-tt self-assigned this Dec 9, 2023
@mreid-tt mreid-tt changed the title TT-RSS: Fix bug in wizard scripts tt-rss: Fix bug in wizard scripts Dec 9, 2023
@mreid-tt mreid-tt merged commit 1c4d5e8 into SynoCommunity:master Dec 9, 2023
17 checks passed
@mreid-tt mreid-tt deleted the tt-rss-update branch December 9, 2023 22:57
@mreid-tt mreid-tt added the status/published Published and activated (may take up to 48h until visible in DSM package manager) label Dec 10, 2023
@mreid-tt
Copy link
Contributor Author

@publicarray, I've identified a bug in a script that I recently implemented across several PHP-type packages I've been working on. I've already addressed the issue by making a minor version update to the package and have pushed the corrected version to the repository. Whenever you have a moment, could you please remove version "20230828-17" from the repo? Thank you!

@publicarray
Copy link
Member

@mreid-tt I have given you more access, so you can delete packages as well

@publicarray
Copy link
Member

publicarray commented Dec 10, 2023

Just be mindfull that becaurse of caching, exepect a 24h delay when removing or publishing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status/published Published and activated (may take up to 48h until visible in DSM package manager)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants