Skip to content
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

Xmp header android #37

Merged
merged 7 commits into from
Jan 10, 2024
Merged

Xmp header android #37

merged 7 commits into from
Jan 10, 2024

Conversation

JangSungChul
Copy link
Collaborator

@JangSungChul JangSungChul commented Dec 19, 2023

issue

adobe情報がhedaderに入ってない
https://github.com/ncdcdev/KuiManagementSystem/issues/2603
#30

変更内容

  • hook処理で移行先のフォルダが存在しない場合、エラーになる問題対応
  • xmpのヘッダーにadobeの情報を入れる処理を追加
  • com.adobe.xmp:xmpcoreのFrameworkを追加

確認したこと

  • 写真撮影後、ヘッダーにadobeの情報が入っていること

スクリーンショット

補足事項

PR時のセルフチェック

  • 関数や変数の命名は一目で分かるものになってますか?
  • コメントは適切な量で、誰が見ても分かるコメントになってますか?
  • PR内容に関するテストは書きましたか?
  • PR内容に付随する各種ドキュメントは一緒に修正しましたか?
  • Issue の完了の定義は満たせていますか?

@JangSungChul JangSungChul self-assigned this Dec 19, 2023
@karuru6225
Copy link
Member

@JangSungChul 試してみました。ビルド中(Build APK(s))を実行中に以下のようなエラーが起きました。

スクリーンショット 2023-12-19 18 18 35

@JangSungChul
Copy link
Collaborator Author

@karuru6225 こちらバージョン変更でビルドエラーの対応できました。

@karuru6225 karuru6225 merged commit d679926 into master Jan 10, 2024
@karuru6225 karuru6225 deleted the fix/xmp_header_android branch January 10, 2024 07:40
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