Skip to content

Add plugin: VCF Contacts #5712

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

Merged
merged 2 commits into from
May 6, 2025
Merged

Add plugin: VCF Contacts #5712

merged 2 commits into from
May 6, 2025

Conversation

broekema41
Copy link
Contributor

I am submitting a new Community Plugin

Repo URL

Link to my plugin: https://github.com/broekema41/obsidian-vcf-contacts/

Release Checklist

  • I have tested the plugin on
    • Windows
    • macOS
    • Linux
    • Android (if applicable)
    • iOS (if applicable)
  • My GitHub release contains all required files (as individual files, not just in the source.zip / source.tar.gz)
    • 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.

Copy link

Hello!

I found the following issues in your plugin submission

Errors:

❌ Please don't use the word obsidian in the plugin ID. The ID is used for your plugin's folder so keeping it short and simple avoids clutter and helps with sorting.
❌ Please don't include Obsidian in the plugin description
❌ Your plugin has a long description. Users typically find it difficult to read a very long description, so you should keep it short and concise.
❌ Unable to find a release with the tag 1.0.0. Make sure that the version in your manifest.json file in your repo points to the correct Github Release.


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 github-actions bot changed the title Update community-plugins.json for VCF Contacts Plugin Add plugin: VCF Contacts Mar 15, 2025
@broekema41 broekema41 closed this Mar 15, 2025
@broekema41 broekema41 reopened this Mar 15, 2025
@ObsidianReviewBot
Copy link
Collaborator

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

Required

[1]:Do not detach leaves with your custom view in onunload, this is an antipattern, see: https://docs.obsidian.md/Plugins/Releasing/Plugin+guidelines#Don't+detach+leaves+in+%60onunload%60

[1][2]: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

[1]:You should avoid assigning styles via JavaScript or in HTML and instead move all these styles into CSS so that they are more easily adaptable by themes and snippets.

[1]:You should not cast this, instead use a instanceof check to make sure that it's actually a file/folder.


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 ObsidianReviewBot added Changes requested Additional review required PR needs to be reviewed by another person, after the currently requested changes have been made and removed Ready for review labels Mar 16, 2025
@broekema41
Copy link
Contributor Author

This afternoon I have updated, created release 2.0.1 resolving all the review bot items.
Hoping for a listing soon. regards, Ro

@broekema41 broekema41 closed this Mar 23, 2025
@broekema41 broekema41 reopened this Mar 23, 2025
@broekema41
Copy link
Contributor Author

Hello @joethei @Zachatoo Noticed that you two have allot on your plate trying to keep the plugin ecosystem going and its quality up. I will await my turn and monitor this pull request on a regular basis. regards, Ro

@broekema41 broekema41 closed this Mar 23, 2025
@broekema41 broekema41 reopened this Mar 23, 2025
@broekema41
Copy link
Contributor Author

Released version 2.0.2 with avatar processing.
https://github.com/broekema41/obsidian-vcf-contacts/releases/tag/2.0.2

@ObsidianReviewBot ObsidianReviewBot added Ready for review and removed Changes requested Additional review required PR needs to be reviewed by another person, after the currently requested changes have been made labels Apr 17, 2025
@ObsidianReviewBot ObsidianReviewBot removed their assignment Apr 17, 2025
@ObsidianReviewBot
Copy link
Collaborator

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

@joethei
Copy link
Collaborator

joethei commented Apr 17, 2025

@joethei joethei self-assigned this Apr 17, 2025
@joethei joethei added Changes requested Minor changes requested PR can be merged after some final changes have been requested and removed Ready for review labels Apr 17, 2025
@broekema41
Copy link
Contributor Author

Thank you for the feedback @joethei

Made corrections for all the items mentioned and released version 2.0.3:
https://github.com/broekema41/obsidian-vcf-contacts/releases/tag/2.0.3

A overview of the file changes can be found at:
https://github.com/broekema41/obsidian-vcf-contacts/pull/3/files

@broekema41
Copy link
Contributor Author

good day @joethei, is there anything I should do here? Just noticed that my comment rectification of the review items was 3 weeks ago. Thanks Roland

@joethei joethei added Changes made and removed Changes requested Minor changes requested PR can be merged after some final changes have been requested labels May 5, 2025
@ObsidianReviewBot
Copy link
Collaborator

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

Required

[1]:You should avoid assigning styles via JavaScript or in HTML and instead move all these styles into CSS so that they are more easily adaptable by themes and snippets.


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 ObsidianReviewBot added Changes requested Additional review required PR needs to be reviewed by another person, after the currently requested changes have been made and removed Ready for review labels May 6, 2025
Copy link

github-actions bot commented May 6, 2025

Hello!

I found the following issues in your plugin submission

Errors:

❌ Could not parse community-plugins.json, invalid JSON. Expected ',' or ']' after array element in JSON at position 612503


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.

@joethei joethei merged commit b5c4c3a into obsidianmd:master May 6, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Additional review required PR needs to be reviewed by another person, after the currently requested changes have been made Changes made Changes requested plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants