Repository navigation
Add DailySounds (audio) - #164
Open
jaydenhertz wants to merge 5 commits into
Open
jaydenhertz wants to merge 5 commits into
jaydenhertz wants to merge 5 commits into
Conversation
Owner
|
Thanks for the clear disclosure and the careful entry. I re-read the licence, home and packs pages today (2026-10-29), and every Evidence quote matches word for word. One change before merge: please drop the tracking parameter from Heads-up: #163 (Soundza) also takes the total from 417 to 418, so whichever merges second will need |
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
We run DailySounds and are submitting it ourselves, per CONTRIBUTING "Submitting your own source". The entry is written as a third party would (facts the site states, no comparisons or ranking language) and starts at
needs-reviewso a maintainer can re-read the licence before it goesactive.New entry:
catalog/audio/dailysounds.md, a free sound effects site (over 1,100 clips, MP3, WAV or OGG per sound, no account). Licence: the DailySounds sound license (custom): commercial use in games and apps allowed, credit optional, no reselling or re-uploading the raw files, no Content ID registration. Notes also record the licence page's own caution about fan-made clips whose titles name films, games or brands, that the WAV and OGG downloads are made from MP3s, and that the WAV packs sold on itch.io are a separate product (not linked).The licence page (https://dailysounds.org/license, "Last updated October 5, 2026"), the home page and the packs page were read on 2026-10-29, the day of submission, and each Evidence line quotes them word for word. The
?ref=fgdaon the url only counts visits from the catalog in our own logs; please drop it if you prefer the bare address. No existing issue or PR mentions DailySounds.Changes: the new entry, its row in
catalog/audio/README.md, and the count updates fromnpm run counts(README.md, catalog/README.md, site/config.json). The files were committed through the GitHub web editor and then checked byte for byte against a local clone where the checks ran.Prepared with AI assistance: an AI assistant working for DailySounds drafted this PR and read every quoted page itself on 2026-10-29.
Type
Checklist
node site/new-entry.mjs <category> <id>or copiedcatalog/TEMPLATE.md(new entries)idis unique across the whole catalog## Evidencehas a dated line quoting itREADME.md, with the same licence as the frontmatternode site/sync-counts.mjsafter adding or removing entriesstacks/README.md; no licence named in a stack's own words (no stack changes)npm test && npm run validatepasses locally (CI runs the fullnpm run check, including the build);npm run checkpasses too (418 entries)