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

'mark as read' fling option #2 #1048

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

JoshAusHessen
Copy link

Second try to add this feature.

Added all feature changes to the changes happening on origin.

@JoshAusHessen
Copy link
Author

Only the Errors, existing in the QuantumBadger:master branch, come up.

@JoshAusHessen JoshAusHessen marked this pull request as ready for review February 5, 2023 09:52
@codeofdusk
Copy link
Contributor

I've opened JoshAusHessen#2.

@JoshAusHessen Could you please merge that into your branch?

@QuantumBadger
Copy link
Owner

Thanks both!

@codeofdusk It's probably best to make your changes as a separate PR in this repo -- I'm planning to review and merge this PR now.

@codeofdusk
Copy link
Contributor

@QuantumBadger Done, as #1049.

@QuantumBadger
Copy link
Owner

@JoshAusHessen Thanks for the PR! I've tested this locally and it doesn't have any effect -- I think pref_behaviour_fling_post_actions hasn't been updated (it looks like pref_menus_post_context_items got updated instead). Would you mind fixing and testing yourself before resubmitting please?

pref_behaviour_fling_post_actions,
pref_behaviour_fling_post_actions_return

the action menu doesnt work properly, but the fling action does
@JoshAusHessen
Copy link
Author

JoshAusHessen commented Feb 5, 2023

So the fling option works.
There is however a issue with the post action menu (I added the 'mark as read' option there as well, just for completions sake).

There are multiple options missing not only the added one:
2023-02-05_21:32:57 2023-02-05_21:33:18
Also the current master branch brakes, when I open a Subreddit or the main page.
So I'm not sure it that problem is because of my changes.
I may have to wait for a stable commit first.

@QuantumBadger
Copy link
Owner

The master branch works fine for me, what's the specific error you're getting?

There are multiple options missing not only the added one

That's expected -- options are hidden if they can't be used (e.g. you can't upvote if you're not logged in). The items are added to the menu in RedditPostActions.showActionMenu().

@JoshAusHessen
Copy link
Author

The shut down.
In the last you see the git status and git log

2023-02-06_21:54:20
2023-02-06_21:54:23
2023-02-06_21:54:25
2023-02-06_21:54:27
2023-02-06_21:54:30
2023-02-06_21:54:55
2023-02-06_21:56:19

error_msg.txt

RedditPostActions.showActionMenu()
@QuantumBadger
Copy link
Owner

@JoshAusHessen It looks like that error is caused by the MARK_READ changes in this PR:

java.lang.IllegalArgumentException: No enum constant org.quantumbadger.redreader.common.PrefsUtility.PostFlingAction.MARK_READ

Something must have gone wrong with the build as that issue shouldn't occur on master.

@JoshAusHessen
Copy link
Author

JoshAusHessen commented Feb 6, 2023

You are too quick for me XD I had some issues with github authentication.
I added the Option to the RedditPostActions.showActionMenu() now both the fling option as well as the PostActionMenu work as intended.
2023-02-06_22:26:48

However I also made a clean pull from https://github.com/QuantumBadger/RedReader.git and tried running it on the emulator. There I got again the error as shown above. When I run my fork on the emulator it works. So maybe that can be ignored.

@JoshAusHessen
Copy link
Author

JoshAusHessen commented Feb 6, 2023

Just saw it is already added with #1049... Anyway thx @QuantumBadger @codeofdusk I'm exited for the next update 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants