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

Reddit Install based app #2613

Closed
4 tasks done
solitary8 opened this issue Jun 25, 2024 · 8 comments
Closed
4 tasks done

Reddit Install based app #2613

solitary8 opened this issue Jun 25, 2024 · 8 comments

Comments

@solitary8
Copy link

What is the name of the app?

Reddit Desktop

Where is the app hosted?

https://github.com/solitary8/Reddit

About the app

Reddit but on Desktop using electron to run it as if it were a desktop app,there are some errors while running the app in the terminal but they don't seem to affect the running of the app

Upload file or Add PR Link

Reddit_Desktop.zip

Confirmations

Copy link
Contributor

A zipfile was found in the body of your issue.
The sha1sum of the zip was: c426100108a03dc99c23d0a10e14ad5fb4ce8dbf

Click to show contents preview

run.sh

#!/bin/bash

cd $HOME/Reddit
electron . >> ~/bashrc 

reddit_setup.sh

#!/bin/bash

sudo apt-get install nodejs npm
sudo npm install -g electron
npm init -y
npm install electron --save-dev
electron . 

@solitary8
Copy link
Author

don't hesitate to ask for the errors that I get while running the app

@solitary8
Copy link
Author

I have already done the install and uninstall script using the pi-apps wizard this is just some of the files that are on the github page,I have also added an icon in the apps list in the pi-apps create an app install script and it works

@Botspot
Copy link
Owner

Botspot commented Jun 25, 2024

Where are the install and uninstall scripts? All I see in your zip file is a reddit_setup.sh and a run.sh script, neither of which Pi-Apps has any idea what to do with.
Furthermore, we do not allow app install scripts to use apt-get directly - they have to use install_packages instead.

Please read our documentation. https://pi-apps.io/wiki/development/Creating-an-app/

@solitary8
Copy link
Author

I'm sorry as I said these are for people that want to install it from github,
I have done the same setup script and done a uninstall script for the app I will upload it shortly

@Botspot
Copy link
Owner

Botspot commented Jun 25, 2024

We also generally don't want to add any website-as-an-app because that would flood Pi-Apps.
Anybody with a web browser can create a desktop version of their favorite websites with a few clicks. We also have the "Web Apps" tool on Pi-Apps, which helps simplify that process further.

I see no advantage to using your approach of launching reddit.com - only drawbacks. Electron needs frequent updating, and it also has numerous bad versions that need to be avoided.

Unless you can explain how your electron-based way of launching reddit.com is truly better than chromium or the Web Apps tool, I'm ready to close this issue.

@theofficialgman
Copy link
Collaborator

fine to close by me
some more reading for @solitary8 if they are interested more about our decision process:
#1791
#2333

@theofficialgman theofficialgman closed this as not planned Won't fix, can't repro, duplicate, stale Jun 25, 2024
@solitary8
Copy link
Author

Great thanks for the additional info

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

No branches or pull requests

3 participants