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

feat(ui): new dialog on adding repository and auto redirection #1025

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

Conversation

IndusAryan
Copy link
Contributor

No description provided.

@fire-light42
Copy link
Collaborator

Explain why this should be added and what it does. I can read the code, but I dont understand the intent behind it.

@IndusAryan
Copy link
Contributor Author

IndusAryan commented Apr 23, 2024

Explain why this should be added and what it does. I can read the code, but I dont understand the intent behind it.

Out of all extensions for every repository only few work, but people press download button and add all of them , even disabled ones, that contributes to huge amount of bloat as well as waste of time, for example an Indian presses download button when adding hexated repo, instead of only 2 hindi extesnions he downloads whole bunch of indonesian, phillipino and s*x extensions that are of no use to her, not only uninstalling them is a waste of time, but produces bloat in extension sheet, in new way we show a reposiotry download succesfully dialog and redirect to that particular repo to download her desired extensions, and ask them that we dont take any responsibility of 3rd party extensions, because nowadays people are using some 3rd party sites that we are not affiliated with, and thus they will download only those with their flags and languages.

this is a win win fr, big fan bhai big fan ❤️‍🔥

@Luna712
Copy link
Contributor

Luna712 commented Apr 23, 2024

For the record I quite like this PR as I accidentally press download all sometimes myself then spend time removing extensions so I do think this is a good idea personally.

@fire-light42
Copy link
Collaborator

I dont see any obvious bugs from reading the code, so if I dont find anything in testing this will be merged.

app/src/main/res/values/strings.xml Outdated Show resolved Hide resolved
@fire-light42
Copy link
Collaborator

This will be merged after further testing.

IndusAryan and others added 2 commits May 26, 2024 01:04
# Conflicts:
#	app/src/main/java/com/lagradost/cloudstream3/MainActivity.kt
setTitle(repositoryName)
setMessage(R.string.download_all_plugins_from_repo)
// won't show "open button" when adding from settings
if (!isExtensionsFragment) {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why??????

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if user is adding a repo manually with add repo button, he doesnt need the go to repo button, because the added repo is visible anyways after dismissal

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nah, removing that option just makes it a waste of a popup then

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what to do then, remove popup here or add button ? without popup we can show a toast of added successfuly, non intrusive also

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

Successfully merging this pull request may close these issues.

None yet

3 participants