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

Wavelog - Small Adjustment #292

Merged
merged 2 commits into from
Nov 17, 2024
Merged

Conversation

HB9HIL
Copy link
Contributor

@HB9HIL HB9HIL commented Nov 16, 2024

Hello together

Thanks to @DonLocke there is now a Wavelog Helper Script available (PR #259) . Thanks for the contribution and supporting our project with that. As maintainer we are really lucky to have such a great community.

I have just two small things which I want to add/change:

  • In PR Update Wavelog icon #282 @havardthom changed the logo so it matches better with the github dark theme. This is totally fine. It would be just better to stay within the Github Ecosystem and don't use the Logo from our Wordpress Page. There is an "icon only" logo available in our repo so it's better to use that.
  • Not officially in our documentation yet, but there will be a new dependencie pretty soon. php-gd will be required upon the next release. I added this in the install script.

Thank you for reviewing and checking this PR.

In the amateur radio world we would say 'Tnx and 73' which means 'Thank you and Best Regards'

@HB9HIL HB9HIL requested a review from a team as a code owner November 16, 2024 23:31
@github-actions github-actions bot added update script A change that updates a script documentation A change to documentation labels Nov 16, 2024
@HB9HIL HB9HIL changed the title Wavelog Wavelog - Small Adjustment Nov 16, 2024
@havardthom havardthom merged commit d3633f9 into community-scripts:main Nov 17, 2024
3 checks passed
@HB9HIL HB9HIL deleted the wavelog branch November 17, 2024 10:18
@havardthom havardthom added website A change to the website and removed documentation A change to documentation labels Nov 17, 2024
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 website A change to the website
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants