violet icon indicating copy to clipboard operation
violet copied to clipboard

Bookmark is not working in artist_info_page

Open seunggil1 opened this issue 1 year ago • 4 comments

Environment

  • version : 1.31.1
  • platform : Android

Issue

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

Steps to Reproduce

  1. 아무 작품 선택
  2. 작가 눌러서 Artist Info Page 접속
  3. 작품 아무거나 선택해서 읽기
  4. 뒤로 돌아와서 상단 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

seunggil1 avatar Apr 14 '24 10:04 seunggil1

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

TaYaKi71751 avatar Apr 18 '24 02:04 TaYaKi71751

@TaYaKi71751

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

seunggil1 avatar Apr 19 '24 21:04 seunggil1

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

violet-dev avatar May 06 '24 05:05 violet-dev

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

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

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

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

seunggil1 avatar May 07 '24 13:05 seunggil1