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

postcapture 卡片上方需顯示為寄件人信箱 而不是收件者信箱 #198

Closed
scott-dt opened this issue Nov 13, 2020 · 1 comment
Assignees
Labels
bug Something isn't working fix/feature verified The fix has been verified by QA workaround Fixed with a workaround
Milestone

Comments

@scott-dt
Copy link

Description

postcapture 卡片上方需顯示為寄件人信箱 而不是收件者信箱

Expected Behavior

  • Expected: postcapture 卡片上方需顯示為寄件人信箱
  • Actual: postcapture 卡片上方需顯示為收件人信箱

Logs

Screenshot_20201113-183725

Environment

  • Version: 0.7.3
  • OS: Android 8.1.0
  • Device: HTC Exodus 1
@scott-dt scott-dt added the bug Something isn't working label Nov 13, 2020
@scott-dt scott-dt modified the milestones: backlog, beta 6.2 Nov 13, 2020
seanwu1105 added a commit that referenced this issue Nov 16, 2020
This is implemented with a BAD architecture. The PostCapture wall only display contents from the server,
and the app does NOT save any cache of PostCaptures. The performance will deteriorate for the huge
ovehead from the Internet. This architecture should be redesign with the new (reliable) storage
implementation and the new data model for the exchange between server and app.
seanwu1105 added a commit that referenced this issue Nov 16, 2020
* Rename app name to Capture and proof detials to capture details. #201 #202

* Show sender on app-post-capture-card. #198

This is implemented with a BAD architecture. The PostCapture wall only display contents from the server,
and the app does NOT save any cache of PostCaptures. The performance will deteriorate for the huge
ovehead from the Internet. This architecture should be redesign with the new (reliable) storage
implementation and the new data model for the exchange between server and app.

* Fix tests.
@scott-dt
Copy link
Author

0.7.5 修正確認
Screenshot_20201116-214028_Capture

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working fix/feature verified The fix has been verified by QA workaround Fixed with a workaround
Projects
None yet
Development

No branches or pull requests

2 participants