Skip to content

Replace More app#1615

Closed
Armored-Dragon wants to merge 9 commits intooverte-org:masterfrom
Armored-Dragon:feature/more/initial_replacement
Closed

Replace More app#1615
Armored-Dragon wants to merge 9 commits intooverte-org:masterfrom
Armored-Dragon:feature/more/initial_replacement

Conversation

@Armored-Dragon
Copy link
Contributor

@Armored-Dragon Armored-Dragon commented Jun 10, 2025

This pull request changes the installed app-more.js script with a new one, which will also allow the installation of third-party repositories. This replacement also provides a more streamlined approach to distributing different versions of the same application. For example, you can have a "stable" version, along with a "testing" version. You chose which version you wish to install from the application details page. This application is backwards compatible with the current community-apps metadata.js file. However this support is hard coded and will not work with other repositories that follow the same format as this one. I don't anticipate this being a problem as the current more app does not allow additional repositories to be added anyways.

This pull request also moves the current app-more.js to the script-archive folder, and updates the default scripts so that this application takes the place of the original app-more.js application.

For testing, you can use my repository and add it as an entry to the list of application repos.
Use this URL to load my repository: https://raw.githubusercontent.com/Armored-Dragon/overte-applications/refs/heads/master/metadata.json

TODO:


Application listing

Application details page

Installed repository list

Application details page (App installed)

Application listing with search

App install selection

@Armored-Dragon Armored-Dragon self-assigned this Jun 10, 2025
@Armored-Dragon Armored-Dragon added enhancement New feature or request work in progress Do not merge yet needs CR This pull request needs to be code reviewed needs QA This pull request needs to be tested NLnet labels Jun 10, 2025
This should explain how to create your own repository.
This now hides apps that have the "appActive" field set to anything other than "true".
@Armored-Dragon Armored-Dragon marked this pull request as ready for review June 10, 2025 23:18
@Armored-Dragon Armored-Dragon removed the work in progress Do not merge yet label Jun 10, 2025
@Armored-Dragon
Copy link
Contributor Author

As I have just made some minor adjustments to the visuals of the app details page, here's what they look like:
image
Application version listing.

image
Application version listing when hovered. It will display the url that will be loaded as a script.

image
The app will now display the version title of the installed version instead of the full version url.

@ada-tv
Copy link
Collaborator

ada-tv commented Jun 10, 2025

Apps aren't marked as installed/uninstalled until the more app is closed and opened again. Maybe the app description text could be a little brighter too? It feels a bit too low contrast for my eyes, at least.

@Armored-Dragon

This comment was marked as outdated.

@ada-tv

This comment was marked as outdated.

@Armored-Dragon

This comment was marked as outdated.

@Armored-Dragon
Copy link
Contributor Author

Apps aren't marked as installed/uninstalled until the more app is closed and opened again.

Turns out it was a simple oversight.

More.js would try to check to see what apps were installed, but it stopped looking after the first installed application.
73bf6e4

@ada-tv
Copy link
Collaborator

ada-tv commented Jun 11, 2025

It works now :>

@AleziaKurdis
Copy link
Contributor

AleziaKurdis commented Oct 4, 2025

Design issue:
Having a list of names without any description is a not helpful.
The user will have to open every tile to figure what it is.
Just a name is clearly not enough to know what it is in every cases.
Have the Repository origin in the list is a secondary importance compare to the description (or at least the first N char of it ... it would be already something.

This is why they do that in the most of the app stores… having a description on the list.

Features to add:

  • Retry mechanism (10 second after the load of the app, retry to launch every installed app that is not running)
  • “NOT RUNNING” Status visible when it is installed but not running
  • "Reload" button on all of the “installed” app (running or not)
  • "Stop" button on the installed app (that doesn’t uninstall but just stop the script to run)

Features to consider adding:

  • An “installed” app filter (we need to see the state in the list… if it is installed or not)
  • A “Not running” installed app filter (we need to see the state in the list… if it is installed but not running)
  • A repository filter on the list

Suggestion for the entries on the list:

  • Icon (Maybe have a generic one for script that are not apps)
  • name
  • status ("installed" (green dot), "installed but not running" (red dot)), no indicator for the non installed. Could be displayed next to the name)
  • description (only the first N char could make a big difference, and keep the layout constant)
  • repository

@AleziaKurdis AleziaKurdis self-assigned this Oct 7, 2025
@AleziaKurdis
Copy link
Contributor

Retaking this project.

@AleziaKurdis
Copy link
Contributor

image001

@AleziaKurdis
Copy link
Contributor

Please CLOSE this PR and consider now the PR#1838 (which retake this but with a partially re-design under the hood.)

@AleziaKurdis AleziaKurdis added the To Be Closed Suggested to be closed. label Oct 9, 2025
@ada-tv ada-tv closed this Oct 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request needs CR This pull request needs to be code reviewed needs QA This pull request needs to be tested NLnet To Be Closed Suggested to be closed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants