Skip to content

daily-wallpaper#8

Merged
ItsLemmy merged 2 commits into
noctalia-dev:mainfrom
nzlov:main
Jul 15, 2026
Merged

daily-wallpaper#8
ItsLemmy merged 2 commits into
noctalia-dev:mainfrom
nzlov:main

Conversation

@nzlov

@nzlov nzlov commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Plugin

  • Id: nzlov/daily-wallpaper
  • New plugin

What it does

Fetches a new daily wallpaper from Bing or NASA.

External dependencies

None

Testing

  • Tested on Niri

Screenshots / Videos

image

Checklist

  • The directory name matches the part of id after the / in plugin.toml exactly.
  • It ships plugin.toml, README.md, thumbnail.webp, and translations/en.json.
  • version follows semver and is bumped in this PR; min_noctalia is the version I tested against.
  • I only added or edited translations/en.json (no machine-translated locales).
  • I did not edit catalog.toml; CI generates it.
  • This PR touches exactly one plugin directory.

Code review attestation

Plugins run as trusted, unsandboxed Luau in the user's session. Confirm:

  • The code is readable and not obfuscated, minified, or generated.
  • It does not download and execute remote code.
  • Every network call, filesystem write, and spawned process is something the description above accounts for.
  • I have the right to publish this code under the license declared in plugin.toml.

fix: 修复每日壁纸预览图

clean: translatiions

fix
@ItsLemmy

Copy link
Copy Markdown
Contributor

The thumbnail does not use the thumbnail generator as it should

Comment thread daily-wallpaper/daily_wallpaper.luau Outdated
Comment thread daily-wallpaper/daily_wallpaper.luau
Comment thread daily-wallpaper/daily_wallpaper.luau Outdated
@nzlov

nzlov commented Jul 14, 2026

Copy link
Copy Markdown
Contributor Author

Thanks for the detailed review. Could you please take another look when you have a chance?

@ItsLemmy

Copy link
Copy Markdown
Contributor

Thanks

@ItsLemmy
ItsLemmy merged commit 6e77640 into noctalia-dev:main Jul 15, 2026
1 check passed
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.

2 participants