obsidian-releases icon indicating copy to clipboard operation
obsidian-releases copied to clipboard

Add better-inline-fields plugin

Open dsarman opened this issue 1 year ago • 2 comments

I am submitting a new Community Plugin

Repo URL

Link to my plugin: https://github.com/dsarman/better-inline-fields

Release Checklist

  • [x] I have tested the plugin on
    • [ ] Windows
    • [x] macOS
    • [ ] Linux
    • [ ] Android (if applicable)
    • [ ] iOS (if applicable)
  • [ ] My GitHub release contains all required files
    • [x] main.js
    • [x] manifest.json
    • [x] styles.css (optional)
  • [x] 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)
  • [x] The id in my manifest.json matches the id in the community-plugins.json file.
  • [x] My README.md describes the plugin's purpose and provides clear usage instructions.
  • [x] I have read the tips in https://github.com/obsidianmd/obsidian-releases/blob/master/plugin-review.md and have self-reviewed my plugin to avoid these common pitfalls.
  • [x] I have added a license in the LICENSE file.
  • [x] 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.

dsarman avatar Jul 30 '22 12:07 dsarman

Hello dsarman!

I found the following errors in your plugin, Commander:

:x: The newly added entry is not at the end, or you are submitting on someone else's behalf. Last plugin in the list is: phibr0/obsidian-commander


This check was done automatically.

github-actions[bot] avatar Jul 30 '22 12:07 github-actions[bot]

As your plugin uses Dataview you should add a notification somwhere, either as a notice or in the settings if Dataview is not installed/enabled.

joethei avatar Aug 10 '22 12:08 joethei

Agree with joethei's comment. It would be useful to add a check when the plugin loads to make sure that dataview is installed.

liamcain avatar Aug 12 '22 18:08 liamcain

Thanks for the approve and for the suggestion. I added a notice to the plugin's settings page if Dataview is not enabled.

dsarman avatar Aug 22 '22 20:08 dsarman