[go: nahoru, domu]
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.

Sign up for GitHub

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

Jump to bottom

Refreshing posts after discarding changes still prompts you about unsaved changes. #25

Closed
aerych opened this issue Jul 31, 2013 · 4 comments
Closed

Refreshing posts after discarding changes still prompts you about unsaved changes. #25

aerych opened this issue Jul 31, 2013 · 4 comments
Labels
[Type] Bug

Comments

@aerych
Copy link
Member
aerych commented Jul 31, 2013

On the Nexus 7.
Steps to reproduce:

  • In the post detail panel, tap to edit a post.
  • Make a change (or not) and hit the back button to return to the post list.
  • Choose "Discard" when prompted to save your changes.
  • Back on the post list, tap to refresh the list.

You're prompted that you have local changes, even though you just discarded them.
@roundhill
Copy link
Contributor

Are you sure you don't have unsaved changes for another post? (Kind of a separate issue, we should probably mark posts that have unsaved changes in the post list)

@aerych
Copy link
Member Author
aerych commented Jul 31, 2013

I believe so. I see the same behavior if I refresh the post list a few times before tapping to edit the post. Since I'm not prompted when refreshing the post list initially I'm assuming I have no posts with unsaved changes. Also this is a clean install so there shouldn't be any drafts lingering around.

@roundhill
Copy link
Contributor

I'm seeing this bug now as well. It is quite annoying :)

@aerych
Copy link
Member Author
aerych commented Aug 5, 2013

Sneaky autosave

mufela added a commit to mufela/WordPress-Android that referenced this issue Dec 5, 2016
@mufela
wordpress-mobile#25
a047abc
AmandaRiu added a commit that referenced this issue Aug 1, 2019
@AmandaRiu
Squashed 'libs/login/' changes from 69e488759c..fc60b518d7
db8d333 
fc60b518d7 Merge branch 'develop' into merge/woocommerce-android/1287
38737cef3b Merge pull request #25 from wordpress-mobile/merge-wpa
d035db2384 Merge commit 'eb8270b278800a6804a2fce6bb63111c0dfa789a' into update-login-lib
4e66d2a380 Move signedin tracking to the google fragment
a1e403a879 Track signed in event on sign up
d983da8c98 Merge pull request #10232 from wordpress-mobile/amanda/login-lib-redirect-support
3bf6e99f15 Merge pull request #24 from wordpress-mobile/merge/WordPress-Android/10255
895ece8fc4 Merge commit '3a2393c33fe8f1f1448591580fc0054a732fac9b' into amanda/login-lib-redirect-support
5a21797a03 Merge pull request #23 from wordpress-mobile/merge/woocommerce-android/1224-amanda-new
8fee8b3660 Updated login and editor libraries to remove right/left attributes when a start/end attribute already exists
2289f9707b Merge commit '43ac7e9468ada491ccd1fcd63cfa1ef0616bd791' into issue/9720-login-subtree-update
b5ac63f082 Fix redundancy in R.string.enter_site_address
bc3903bd29 Update login_invalid_site_url
5d1b930343 Add tests for LoginSiteAddressValidator
ae203df2ec Delete unused R.string.login_empty_site_url
763eeb9490 Replace login_invalid_site_url
1032574f3f Do not report an error if the text is empty
1fa189853a Site Address: Show an error message after 2 sec
e869735555 Do not react to keyboard enter if site is invalid
8d50ab92c6 Move LoginSiteAddress validation to a class
3a9a5c8566 Merge commit 'e3144c2c11b0c982eadb2478cf6926aa40c127e6' into amanda/login-lib-redirect-support
3b1db07c38 Merge commit '615ebab45e1bc3d0cdd3a9da060026fef704d3a8' into amanda/test_git_subtree
a8f234de25 Merge pull request #10150 from wordpress-mobile/amanda/woo-login-test-wp-2
43ac7e9468 Merge pull request #22 from wordpress-mobile/merge-wca
d4d717cf16 Send redirectUrl to listener and strip protocol before saving
6dbf72672a Updated login and editor libraries to remove right/left attributes when a start/end attribute already exists
979ddf4aec Merge commit '827a47189094a6bc7800eaccc3ed069841251373' into sync-login-lib
df3c11ed09 added textAlignment property to secondary button in login
da783ea3c7 Login screens: added textAlignment property to labels
6d486bbf6b Merge pull request #21 from wordpress-mobile/merge-wpa-develop
57a2d00b14 Update login lib standalone FluxC hash
0ea2405def Merge commit 'ad485b27b26ffb38b8718940b0b7556b902dc28c' into sync-login-lib
6df15ab86b Update 2FA unavailable API error message
eefd72f317 Show error and continue login on OAuth client 2FA error
578dca7f29 Update androidx dependencies
1d09d1536d Fix AndroidX import order
48df84638f Fix import ordering for androidx
d3dc35035a Migrate to AndroidX
78c5307f7a Merge pull request #16 from wordpress-mobile/fix_crash_in_login_email_fragment
4d6c83f111 Merge pull request #9890 from wordpress-mobile/update-dagger-version-to-v2.22.1
edd8684ec4 Merge pull request #18 from wordpress-mobile/new-wpandroid-changes
9cbf3f17ff Merge pull request #9885 from wordpress-mobile/update-login-library
71729e5e76 Merge pull request #17 from wordpress-mobile/issue/glide-4.9
0ddf4dcdbd Update Glide version to 4.9.0
f7860b9861 Updated Dagger in the login library
aabdedf672 Merge commit '099b5a011d32283eb6c79b0e599b5272a597e849' into update-login-library
1eb7f4f156 Updated Glide to 4.9.0
a32cc8c09d Add crash fix to SignupEmailFragment
08109fa90f Adds non-null check to onActivityResult
664b67f62d Fix  #9826: make sure we Play Services are available before running the email hint picker
94a0ae2a39 Merge pull request #9743 from wordpress-mobile/feature/manage_insights_readonly
7cad17e264 Fix a few ContentDescription lint errors
9f171b55d0 Merge branch 'feature/manage_insights_readonly' into issue/9464-manage-insights
663a168903 Add the missing drawable resources

git-subtree-dir: libs/login
git-subtree-split: fc60b518d7c72864f6df54546d640d0e2fbb8673
jd-alexander added a commit that referenced this issue Jan 12, 2020
@jd-alexander
Squashed 'libs/utils/' changes from 4c6e8c9234..a15de7604a
90001f8 
a15de7604a Merge pull request #25 from wordpress-mobile/deprecate-buggy-date-time-utils-methods
5026dc0904 Mark DateTimeUtils.localDateToUTC and DateTimeUtils.nowUTC as deprecated
58cd5bf239 Mark DateTimeUtils.localDateToUTC and DateTimeUtils.nowUTC as deprecated
4360e5d41f Merge pull request #23 from wordpress-mobile/update-bintray-config
48277981d8 Bump version to 1.23
81f224bd56 Fix Javadoc lint build error when releasing to Bintray
240f2492fa Update Bintray plugin to latest
b6abdcae2a Merge pull request #22 from wordpress-mobile/merge-wpa
8260cdce57 Update style and lint configs
7c08103ff2 Update Gradle wrapper version
9666183e69 Merge commit 'a4a756b60bcbf144ed8527f693c4e974fa6e365d' into subtree-updates-v3
cdeb3eb7f1 Merge pull request #9215 from wordpress-mobile/feature/update-support-lib-28
e8dbd8f404 Revert update of targetSdkVersion
f7014365ea Update supportLib version to 28.0.0
b3ad4c795e Fix low hanging deprecation warnings
b028488305 Remove all instances of LOCATION
bd194edb49 Merge pull request #9114 from wordpress-mobile/issue/8177-applog-testing
e081d898e7 Merge pull request #9044 from wordpress-mobile/gradle-4-10
2ba6f52024 Move fake AppLog(the one for testing) from utils to the main project
3f74b509d0 Create copy of AppLog for unit testing purposes
78d9bfbf13 fixed merge conflict
b3b6665bfd fixed merge conflict
c2fa689153 Update to Gradle 4.10.3/Android Gradle plugin 3.2.1
62b03547dc Merge remote-tracking branch 'origin/develop' into feature/master-site-creation
f55c4c33b1 Removed duplicated dots from image URL
818c1691c7 Rename NewSiteCreationService props
ab051f9083 removed unused imports
0ea8d2aab7 moved unused getDate() method from MediaUtils to getFormattedDateForLastModified() in PostUtils
6e4cecb37b Make the progress of NewSiteCreationService indeterminate
4434057372 Merge branch 'develop' into feature/master-site-creation
e7e89ad67e Fix SiteCreation segment icon color tint
2259033733 Merge branch 'develop' into feature/deep-link-main-activity
0671724495 Merge remote-tracking branch 'origin/develop' into feature/deep-linking-improvements
8f5238e5ef Reorder repositories in build.gradle to fix Gradle
4f18db4eb3 Reorder repositories in build.gradle to fix Gradle
e695cb4a6b Refactor isDeepLinking to be outside MyProfileActivity
d85b6dbb16 Remove unused WPImageGetter
44d63ede88 Add a new type of log tag for pages
5b4a4f09ae Move the logging wrapper class to FluxCUtils for convenience
c7c1ff782a Add Crashlytics logging to make sure we're keeping track of problems uploading media
d12f32a4a7 Remove coroutines and replace them with standard event driven architecture
5580689d26 Remove unused code - ImageUtils.getThumbnail

git-subtree-dir: libs/utils
git-subtree-split: a15de7604a34a9e989884fb84b8c9adfaeeb97b0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
[Type] Bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@roundhill @aerych