Skip to content

add logseq-md-convert#786

Open
shpeng1996 wants to merge 3 commits intologseq:masterfrom
shpeng1996:master
Open

add logseq-md-convert#786
shpeng1996 wants to merge 3 commits intologseq:masterfrom
shpeng1996:master

Conversation

@shpeng1996
Copy link
Copy Markdown

Submit a new Plugin to Marketplace

Plugin GitHub repo URL: https://github.com/shpeng1996/logseq-md-convert

GitHub releases checklist

  • a legal package.json file.
  • a valid CI workflow build action for Github releases. (theme plugin for this).
  • a release which includes a release zip pkg from a successful build.
  • a clear README file, ideally with an image or gif showcase. (For more friendly to users, it is recommended to have English version description).
  • a license in the LICENSE file.

⚠️ The new incoming plugin recommended fields in manifest.json:
supportsDB - [optional] Whether the plugin supports database graph.
supportsDBOnly - [optional] Whether the plugin only supports database graph.

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 066a2ca8a5

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@@ -0,0 +1,9 @@
{
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Rename plugin manifest to manifest.json

The plugin metadata file is added as menifest.json, but marketplace tooling only loads packages/<id>/manifest.json (see scripts/build.mjs where mfp is hardcoded to manifest.json and missing files are treated as package errors). In this state, logseq-md-convert will be skipped during build/index generation and won’t appear in produced marketplace data even though the package directory exists.

Useful? React with 👍 / 👎.

Copy link
Copy Markdown
Author

@shpeng1996 shpeng1996 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fix manifest.json typo

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.

1 participant