-
Notifications
You must be signed in to change notification settings - Fork 587
Add LinkedIn plugin by abhishek #1890
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Please see the errors in the CI |
crazywoola
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
❌ Request Changes
Decision: Request changes
Failed Checks
| Check | Status | Detail |
|---|---|---|
Icon validation |
❌ Fail | icon contains template placeholder marker. |
Full Check Results
| Check | Status | Detail |
|---|---|---|
Manifest author |
✅ Pass | author is valid. |
Icon validation |
❌ Fail | icon contains template placeholder marker. |
Version check |
✅ Pass | version 0.0.2 is available. |
README language |
✅ Pass | README.md CJK ratio=0.0% (zh=0, en=1046, allowed_zh<=0) |
Dependency install |
✅ Pass | requirements installed successfully. |
dify_plugin version |
✅ Pass | dify_plugin version 0.7.1 satisfies >= 0.5.0. |
Install test |
✅ Pass | plugin install test passed. |
Packaging test |
✅ Pass | packaging check passed. |
Required Fixes
- Icon validation: icon contains template placeholder marker.
Please address these issues and push an update.
Resolved the Icon validation
|
Hi @crazywoola 👋 |
crazywoola
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Review Results
| Check | Status | Required Action |
|---|---|---|
| single_difypkg | ✅ pass | None |
| pr_language | ✅ pass | None |
| manifest | ✅ pass | None |
| icon | ✅ pass | None |
| version_exists | ✅ pass | None |
| readme_language | ✅ pass | None |
| dify_plugin_version | ❌ no pass | Fix: dify_plugin version 0.4.0 < 0.5.0 |
Next steps
Please address the failing checks and push fixes.
crazywoola
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Re-reviewed: dify_plugin>=0.4.0 is valid for current pre-check workflow; prior version failure was a false positive.
Plugin Submission Form
1. Metadata
2. Submission Type
3. Description
4. Checklist
5. Documentation Checklist
Please confirm that your plugin README includes all necessary information:
6. Privacy Protection Information
Based on Dify Plugin Privacy Protection Guidelines:
Data Collection
Privacy Policy