Bookmark is not working in artist_info_page

seunggil1 commented 7 months ago

Environment

version : 1.31.1
platform : Android

Issue

Artist Info Page에서 특정 상황에 Bookmark 버튼이 작동하지 않습니다.

Steps to Reproduce

아무 작품 선택
작가 눌러서 Artist Info Page 접속
작품 아무거나 선택해서 읽기
뒤로 돌아와서 상단 Artist 옆 Bookmark 버튼(하트 버튼 클릭) -> 작동 안됨

Detail Info

artist_info_page의 GestureDetector 작동 잘하다가, 작품 조회 이후 작동이 안되는 것 같습니다. (디버깅을 할 수는 없어서 그냥 추정중입니다)

https://github.com/project-violet/violet/blob/896add9d84240d48f783d0ef07edf4eb0da45267/lib/pages/artist_info/artist_info_page.dart#L391-L445

Etc

직장인이라 많은 기여가 힘들지만, 개발 과정에 도움을 드리고 싶은데, 개발을 위해서는 아래 두 파일이 필요합니다. server/wsalt.dart, server/wsalt.dart api 요청을 위한 v-token, v-valid 정보가 있는 파일로 추정되는데 관련 정보를 제공해주실 수 있으신가요? 보안 문제로 곤란하시면 편하게 거절해주세요.

감사합니다.

@violet-dev

TaYaKi71751 commented 7 months ago

run this on shell :

patch -p1 << EOF
diff --git a/preprocess-android.py b/preprocess-android.py
index 74a339b2..50c94a9e 100644
--- a/preprocess-android.py
+++ b/preprocess-android.py
@@ -108,5 +108,5 @@ for root, subdirs, files in os.walk('./'):
         elif filename.endswith(".yaml"):
             process_yaml(root + '/' +  filename)

-# create_dummy_valid('./lib/server/salt.dart')
-# create_dummy_valid('./lib/server/wsalt.dart')
+create_dummy_valid('./lib/server/salt.dart')
+create_dummy_valid('./lib/server/wsalt.dart')
EOF

run this on shell :

python preprocess-android.py

run this after all on shell :

patch -p1 << EOF
diff --git a/preprocess-android.py b/preprocess-android.py
index 74a339b2..50c94a9e 100644
--- a/preprocess-android.py
+++ b/preprocess-android.py
@@ -108,5 +108,5 @@ for root, subdirs, files in os.walk('./'):
         elif filename.endswith(".yaml"):
             process_yaml(root + '/' +  filename)

-create_dummy_valid('./lib/server/salt.dart')
-create_dummy_valid('./lib/server/wsalt.dart')
+# create_dummy_valid('./lib/server/salt.dart')
+# create_dummy_valid('./lib/server/wsalt.dart')
EOF

seunggil1 commented 7 months ago

@TaYaKi71751

That's an interesting way to do it, I didn't expect that. I've got it working now, thanks!

violet-dev commented 6 months ago

로컬에서 재현이 안되네요 ㅠㅠ

seunggil1 commented 6 months ago

테스트 환경 : Galaxy S9+ (Android 10) Xiaomi 13 pro (Android 14)

영상으로 녹화해서 드립니다(터치부분이 표시됩니다)

https://github.com/project-violet/violet/assets/38664481/0d096c08-ab49-413d-93e9-22346f0e0405

개발환경에서는 오류로 아직 실행을 못하고 있어서, 해결 후에 개발환경에서 한번 테스트 해보겠습니다.

project-violet / violet