Skip to content

New branch - #1

Closed
IKcoding-jp wants to merge 77 commits into
mainfrom
new_branch
Closed

New branch#1
IKcoding-jp wants to merge 77 commits into
mainfrom
new_branch

Conversation

@IKcoding-jp

Copy link
Copy Markdown
Owner

No description provided.

…ィードバック画面やサブスクリプション画面のUIを改善し、コードの可読性を向上させました。
…、ユーザーが解約後に状態を確認できるようにしました。さらに、解約処理のロジックを見直し、解約済みフラグを適切に管理するようにしました。
- クローズドテスト(0.8.5+40)から製品版初回リリース(1.0.0+41)へ
- デバッグログ制御システムの実装完了
- 本番環境でのログ無効化対応
- セキュリティとパフォーマンスの向上
✨ 更新内容:
- アプリ概要を製品版に適した内容に更新
- カメラ機能とAI認識機能を主な機能に追加
- QRコードスキャン機能を追加
- リリース履歴にv1.0.0を追加
- Flutterバージョンを3.35.3に更新
- 製品版初回リリースの詳細な機能説明を追加

🎯 製品版リリース対応完了
…tivity.ktでWebViewの設定と初期化を実装しました。また、広告のリクエスト設定を更新し、バナー広告とインタースティシャル広告の読み込み時にWebViewの初期化を待つ処理を追加しました。デバッグログも強化し、エラー時のトラブルシューティング情報を充実させました。
@github-advanced-security

Copy link
Copy Markdown
Contributor

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

Comment thread .github/workflows/firebase-hosting-merge.yml Fixed
IKcoding-jp and others added 6 commits September 20, 2025 15:41
…な長押し操作のセクションを削除しました。UIの改善を行い、無料トライアル残り日数の表示を新たに追加しました。
…n permissions

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Potential fix for code scanning alert no. 1: Workflow does not contain permissions
cursor[bot]

This comment was marked as outdated.

…oショッピングサービス関連のコードを削除しました。バナー広告の読み込み処理を改善し、プレミアム状態に応じた広告表示のロジックを強化しました。
…素化し、データプロバイダー内の不要なコードを削除しました。さらに、いくつかのウィジェットの装飾に関する修正を行いました。
cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

…のSnackBar表示を`mounted`チェックでラップしました。不要な通知サービス、ストア準備サービス、税関連ユーティリティ、ユーザー表示サービスを削除しました。
cursor[bot]

This comment was marked as outdated.

…や機能を削除し、UIの改善を行いました。広告表示のロジックをプレミアム状態に基づいて強化しました。
…ドを実装し、メイン画面でのタブ編集ダイアログに共有機能のスイッチとタブ選択リストを追加しました。さらに、予算ダイアログでの共有グループ内の予算同期機能を実装しました。
cursor[bot]

This comment was marked as outdated.

Comment thread lib/models/item.dart
this.productUrl,
this.imageUrl,
this.storeName,
this.timestamp,

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Bug: CopyWith Methods Ignore Explicit Nulls

The copyWith methods in both Item and Shop models use null coalescing (??) for newly added nullable fields. This means passing null (or an empty list for List types) to copyWith for these fields will not clear them, as it defaults back to the existing value instead of setting the new explicit null or empty list.

Additional Locations (1)

Fix in Cursor Fix in Web

debugPrint('ユーザーがサインインをキャンセルしました');
return null;
}
final GoogleSignInAccount googleUser = await _googleSignIn.authenticate();

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Bug: Google Sign-In Method Mismatch

The signInWithGoogle method attempts to call _googleSignIn.authenticate(), but this method is not available on the GoogleSignIn object, leading to a runtime error. The intended method is _googleSignIn.signIn(), which returns a nullable GoogleSignInAccount that requires a null check.

Fix in Cursor Fix in Web

Comment thread pubspec.yaml
flutter_launcher_icons:
android: "launcher_icon"
ios: true
image_path: "assets/icon/icon.png"
image_path: "assets/launcher_icon/icon.jpg"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Bug: Launcher Icon Missing in Flutter Assets

The flutter_launcher_icons image_path was updated to assets/launcher_icon/icon.jpg. While the flutter assets section includes the assets/launcher_icon/ directory, the specific icon.jpg file isn't explicitly listed. This could cause the launcher icon to be missing or fail to build.

Fix in Cursor Fix in Web

@@ -61,16 +61,37 @@ class CloudFunctionsService {
Future<Map<String, dynamic>> analyzeImage(String imageUrl) async {
try {
debugPrint('🖼️ 画像解析開始: $imageUrl');
final preview =
imageUrl.length > 50 ? imageUrl.substring(0, 50) : imageUrl;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Bug: Substring Length Error

An imageUrl.substring(0, 50) call can cause a runtime IndexError when imageUrl is shorter than 50 characters.

Fix in Cursor Fix in Web

content:
'1. 本アプリでは、買い物リスト作成のため、商品の棚札を撮影して商品名と価格を読み取る機能を提供いたします。\n'
'1. 本アプリでは、買い物リスト作成のため、商品の値札を撮影して商品名と価格を読み取る機能を提供いたします。\n'

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Bug: Title Mismatch Causes Content Confusion

Section 6's title changed from "棚札読み取り機能" to "値札読み取り機能", but its content still refers to "棚札", creating an inconsistency.

Fix in Cursor Fix in Web

onProgress?.call(OcrProgressStep.completed, 'ChatGPT解析完了');
debugPrint(
'✅ ChatGPT解析成功(シンプル版): name=${result.name}, price=${result.price}');
return result;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Bug: Class Unavailability Despite Import

The ChatGptService class isn't available when final chat = ChatGptService(); is called, which leads to a runtime error. This happens even though the chatgpt_service.dart import is still present in the file.

Fix in Cursor Fix in Web

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