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や信ぴょう性チェック対応 #29

Merged
merged 46 commits into from
Oct 27, 2023
Merged

Conversation

JangSungChul
Copy link
Collaborator

@JangSungChul JangSungChul commented May 26, 2023

issue

https://github.com/ncdcdev/KuiManagementSystem/issues/2547

変更内容

  • xmp用pods情報追加
  • 信ぴょう性チェック用framework追加

確認したこと

  • xmpデータが写真に登録されること
  • 信ぴょう性チェック用データが追加されること

スクリーンショット

補足事項

@JangSungChul JangSungChul changed the title xmpテスト XMPや信ぴょう性チェック対応 Jun 21, 2023
@JangSungChul JangSungChul requested a review from karuru6225 June 21, 2023 09:15
@@ -587,12 +601,25 @@ extension CameraViewController: AVCapturePhotoCaptureDelegate {
let data = resizedImage!.jpegData(compressionQuality: 1.0)
if let jpegData = data {
// let base64String = jpegData.base64EncodedString(options: .lineLength64Characters)
guard let photoInfo = photoInfo else { return }
guard let imageDataEmbedMetaData = ElectronicBlackBoardManager.createImageEmbeddedMetaData(from: jpegData, photoInfo: photoInfo,imageDescription: "test image",model: model(),software: "TPR2") else {
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

imageDescriptionsoftwareの確認が必要

@JangSungChul
Copy link
Collaborator Author

JangSungChul commented Jun 23, 2023

残件

  • JS側との連携テスト
  • XMPの詳細情報:imageDescription、softwareの確認が必要

@rsogo
Copy link
Contributor

rsogo commented Jun 23, 2023

@JangSungChul

XMPの詳細情報:imageDescription、softwareの確認が必要

こちらは何を入れたら良いか不明ということでしょうか?

@JangSungChul
Copy link
Collaborator Author

@rsogo

こちらは何を入れたら良いか不明ということでしょうか?

はい、1つ上にその箇所についてコメントしております。

@karuru6225
Copy link
Member

@JangSungChul software に 以下の用に入れればいいみたいです。

スクリーンショット 2023-09-04 13 08 54

softwareはプラグイン外部から指定できるようにしたいです。
指定しない場合は TPR2 3.1.1 等に設定されてるとありがたいです。

@JangSungChul
Copy link
Collaborator Author

JangSungChul commented Sep 5, 2023

@karuru6225 こちらPlugin側修正&動作確認しました。

  • 新しく追加されたパラメータ
    version
  • 指定しない場合
    スクリーンショット 2023-09-05 17 36 37

@karuru6225
Copy link
Member

@JangSungChul 今の使い方は
blackboard.camera.getPicture(base64, isNeedBlackBoard, blackboardViewPriority, JSON.stringify(jcomsiaObj), resolve, reject);
って感じですけど、
blackboard.camera.getPicture(base64, isNeedBlackBoard, blackboardViewPriority, JSON.stringify(jcomsiaObj), 'TPR2 x.x.x', resolve, reject);
って使い方もできるようになる、という認識で正しいですか?

@JangSungChul JangSungChul merged commit f1cd1e6 into master Oct 27, 2023
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.

3 participants