Skip to content

Bug fix: Paymenter #1917

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

Merged
merged 2 commits into from
Feb 1, 2025
Merged

Conversation

opastorello
Copy link
Contributor

✍️ Description

The Debian system version was written with the first letter capitalized, causing a bug during installation, changed from Debian to debian. The way to call the application name was written incorrectly, it was APP and it should have been APPLICATION.



✅ Prerequisites

The following steps must be completed for the pull request to be considered:

  • Self-review performed (I have reviewed my code to ensure it follows established patterns and conventions.)
  • Testing performed (I have thoroughly tested my changes and verified expected functionality.)

🛠️ Type of Change

Please check the relevant options:

  • Bug fix (non-breaking change that resolves an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change unexpectedly)
  • New script (a fully functional and thoroughly tested script or set of scripts)

📋 Additional Information

I noticed that the header is not being displayed, I didn't identify anything incorrect, the paymenter file is in headers and the function is called.

The Debian system version was written with the first letter capitalized, causing a bug during installation, changed from Debian to debian.
The way to call the application name was written incorrectly, it was APP and it should have been APPLICATION
@opastorello opastorello requested a review from a team as a code owner February 1, 2025 03:40
@github-actions github-actions bot added the update script A change that updates a script label Feb 1, 2025
@michelroegl-brunner michelroegl-brunner merged commit a0d7c5d into community-scripts:main Feb 1, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
update script A change that updates a script
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants