-
-
Notifications
You must be signed in to change notification settings - Fork 607
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
docs(charts): Replace broken links for Paperless-NGX/Jellyfin #12487
Conversation
I'm new to this. Is it an issue that I didn't increase the version? All I changed is the documentation. If I need to change the version, I don't know how to. |
Don't worry, not necessary, it's more if you change the actual code for other files (especially the values or templates that could affect running of the chart). However the lint CI test will make it fail anyways. Once someone else approves it'll get merged but won't be seen until catalog gets pushed again |
Also @stavros-k can you add @aamirazad as a contributor for docs with @all-contributors, thanks! |
@all-contributors please add @aamirazad for docs |
Adds @aamirazad as a contributor for doc. This was requested by stavros-k [in this comment](#12487 (comment)) [skip ci] --------- Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
I've put up a pull request to add @aamirazad! 🎉 |
This PR is locked to prevent necro-posting on closed PRs. Please create a issue or contact staff on discord if you want to further discuss this |
Description
⚒️ Fixes links on jellyfin and paperless-ngx charts.
⚙️ Type of change
🧪 How Has This Been Tested?
N/A
📃 Notes:
✔️ Checklist:
➕ App addition
If this PR is an app addition please make sure you have done the following.
icon.png
Please don't blindly check all the boxes. Read them and only check those that apply.
Those checkboxes are there for the reviewer to see what is this all about and
the status of this PR with a quick glance.