Skip to content

Fix issues with image2card.sh #520

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 1 commit into from
Jan 2, 2025
Merged

Conversation

jmencak
Copy link
Contributor

@jmencak jmencak commented Jan 1, 2025

Changes:

@jmencak
Copy link
Contributor Author

jmencak commented Jan 1, 2025

The invalid json tripped me up for at least 2 hours, hope this helps someone.

Changes:
  - Fix invalid json generated by image2card.sh (s/,/:/)
  - Make sure that image conversion works when supplied images
    are not JPG.
  - Adjust comments based on the current documentation:
    https://owntracks.org/booklet/features/card/#shell-script
@jpmens jpmens merged commit 02dd929 into owntracks:master Jan 2, 2025
@jpmens
Copy link
Member

jpmens commented Jan 2, 2025

Merged, thank you! (and thanks for the -q 2 in the comment)

jpmens added a commit that referenced this pull request Mar 19, 2025
- NEW: support for inline image data in a POI location publishes
- FIX: image2card.sh was creating invalid JSON (#520)
- UPD: deprecate google maps from vmap.html (#512)
- FIX: install unit file into /etc/systemd/system/ot-recorder.service during fpm (#514)
- FIX: fix permissions on files installed to share/docs/ot-recorder (#513)
- FIX: contrib/faces/github2card.py require TID
- FIX: contrib/faces/gravatar2card.sh require TID and replace MD5 by SHA256 for Gravatar
- FIX: contrib/faces/image2card.sh asks for TID to produce CARD
- UPD: changed 'convert' in image2card.sh to 'magick'
- FIX: consolidate raspbian/debian fpm-make invocations
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.

2 participants