Add plugin: Syrinscape Online Player

[scooper4711]

Syrinscape is a TTRPG tool for ambient sounds. This plugin will allow GMs to control the Syrinscape online player from inside of notes, triggering moods and one-shot sounds.

I am submitting a new Community Plugin

Repo URL

Link to my plugin: https://github.com/scooper4711/obsidian-syrinscape

Release Checklist

• I have tested the plugin on
  • Windows
  • macOS
  • Linux
  • Android (if applicable)
  • iOS (if applicable)
• My GitHub release contains all required files
  • main.js
  • manifest.json
  • styles.css (optional)
• GitHub release name matches the exact version number specified in my manifest.json (Note: Use the exact version number, don't include a prefix v)
• The id in my manifest.json matches the id in the community-plugins.json file.
• My README.md describes the plugin's purpose and provides clear usage instructions.
• I have read the developer policies at https://docs.obsidian.md/Developer+policies, and have assessed my plugins's adherence to these policies.
• I have read the tips in https://docs.obsidian.md/Plugins/Releasing/Plugin+guidelines and have self-reviewed my plugin to avoid these common pitfalls.
• I have added a license in the LICENSE file.
• My project respects and is compatible with the original license of any code from other plugins that I'm using.
  I have given proper attribution to these other projects in my README.md.

[github-actions]

Hello!

I found the following issues in your plugin submission

Errors:

❌ Please don't include Obsidian in the plugin description
❌ The authorUrl field in your manifest should not point to the GitHub repository of the plugin.

---

^{This check was done automatically. Do NOT open a new PR for re-validation. Instead, to trigger this check again, make a change to your PR and wait a few minutes, or close and re-open it.}

[github-actions]

Hello!

I found the following issues in your plugin submission

Errors:

❌ Please don't include Obsidian in the plugin description

---

^{This check was done automatically. Do NOT open a new PR for re-validation. Instead, to trigger this check again, make a change to your PR and wait a few minutes, or close and re-open it.}

[github-actions]

Hello!

I found the following issues in your plugin submission

Errors:

❌ Could not parse community-plugins.json, invalid JSON. Unexpected string in JSON at position 442009

---

^{This check was done automatically. Do NOT open a new PR for re-validation. Instead, to trigger this check again, make a change to your PR and wait a few minutes, or close and re-open it.}

[ObsidianReviewBot]

Thank you for your submission, an automated scan of your plugin code's revealed the following issues:

Required

[1]:Using innerHTML, outerHTML or similar API's is a security risk. Instead, use the DOM API or the Obsidian helper functions: https://docs.obsidian.md/Plugins/User+interface/HTML+elements

---

Optional

[1]:Casting to any should be avoided as much as possible.

---

Do NOT open a new PR for re-validation.
Once you have pushed all of the required changes to your repo, the bot will update the labels on this PR within 6 hours.
If you think some of the required changes are incorrect, please comment with /skip and the reason why you think the results are incorrect.

[ObsidianReviewBot]

Changes requested by bot have been made, assigning human for additional review.

[scooper4711]

@joethei - I keep rebasing to eliminate merge conflicts in the PR in order to make your review easier. Is that something you want/expect me to do? Or does that make your job more difficult?

[joethei]

You don't need to keep rebasing, I will do that before merging.

[joethei]

authToken: 'insert-your-auth-token-here',
This should not be the default, it should be empty by default.

const response = await fetch(apiUrl, {
Use the requestUrl function from the Obsidian API instead, it will handle some things like CORS automatically.

containerEl.createEl('h2', { text: 'Syrinscape Plugin Settings' });
Don't add a top-level heading in the settings tab, such as "General", "Settings", or the name of your plugin.

.setName('Auth Token'), .setName('Cache Age'), .setName('Trigger Word'), .setName('Clear Remote Links'), .setButtonText('Clear Remote Links')
Use sentence case in UI

[scooper4711]

@joethei - do I remove the 'changes requested' labels, or do you? I have addressed your comments, closed and re-opened the MR and thought perhaps the bot would make the changes.

edit: I think I discovered the answer. I don't have access to remove the labels. :D

[joethei]

The bot rescans every couple of hours, and will update the labels if all of the required changes have been made.