Rewrite of the website

[kazam0180]

I have looked through it fixed the issues I found. You might want to check FAQ and Wiki page @ivan-hc

[kazam0180]

Some things that need to be done:

• Add APPIMAGE URLs from old markdown files to the sources field in the new json files
• Add json parsing support in AM, AppGallery (and vappman too?)
• Add pla-install URI handling in AM-GUI and AppGallery

The CONTRIBUTING.md is on it's way. Did I miss something?

[ivan-hc]

@andy5995 can you review this?

We also talked about this on Discord.

[kazam0180]

added source urls from both UNOFFICIAL APPIMAGE: and APPIMAGE: lines in some apps

[andy5995]

@ivan-hc I will try.

In my opinion, if a lot of changes are needed for a website rewrite, then it normally should be done in smaller chunks so changes can be efficiently reviewed by the maintainer. If this were my project, I'd suggest a contributor submit the first required stage in a pull request. After that is reviewed and merged, then a "stage 2" commit and PR.

Why are over 3000 files getting changed in this single pull request? The site is normally generated using tools and scripts from the AM repo. If the existing site generator can't write the proposed changes, then my first thought would be a single converter script is needed in the AM repo that will implement the proposed changes to all the appimage entries.

I can't think of any more feedback right now, but if I do, I'll post here. And maybe I'll be able to review more once my questions are answered.

[ivan-hc]

@andy5995 I completely agree. Changes like this need to be analyzed in gradual phases. @coyoteclan here are too many changes to analyze.

[kazam0180]

Why are over 3000 files getting changed in this single pull request? The site is normally generated using tools and scripts from the AM repo. If the existing site generator can't write the proposed changes, then my first thought would be a single converter script is needed in the AM repo that will implement the proposed changes to all the appimage entries.

This is a catalogue, it's expected. The generator script and workflow is updated. It does generate the new website. But I had to test it so I enabled the workflow and ran it. You can see individual commits.
Of course I had to convert the existing apps manually first to retain their data. But new apps and stats are generated like they were previously.

[andy5995]

That's for an example. The original contributing.md said you can add extra buttons. So I included a way to define extra buttons.

Oh, I see now. :)

[andy5995]

@andy5995 I completely agree. Changes like this need to be analyzed in gradual phases. @coyoteclan here are too many changes to analyze.

This general guide has more information about "best practices" https://www.deployhq.com/blog/the-perfect-pull-request-best-practices-for-collaborative-development

[andy5995]

@andy5995 @ivan-hc done.

this app info (AM can use this) gets converted to this json and shows as this page

check it out, check other parts of the website if you have the time, if there are changes to be made, i will do it. Then i will remove the 3369 apps generated and replace them with old md files for PR review

@coyoteclan Please post the screenshots here instead of the links, then remove all the generated artifacts, update the PR. Then we will be able to review the code that changes the website infrastructure.

[kazam0180]

@andy5995 @ivan-hc done.
this app info (AM can use this) gets converted to this json and shows as this page
check it out, check other parts of the website if you have the time, if there are changes to be made, i will do it. Then i will remove the 3369 apps generated and replace them with old md files for PR review

@coyoteclan Please post the screenshots here instead of the links, then remove all the generated artifacts, update the PR. Then we will be able to review the code that changes the website infrastructure.

How apps are defined by contributors or "dumb" apps generated by sync with AM workflow:

This info file is converted to json and served on the deployed site:

And the app.html page renders the json data like this:

real app example, virtualbox:

[kazam0180]

thanks for the article link. i will do it now

[kazam0180]

@ivan-hc i noticed now i have missing categories. you pointed it out before but i didn't pay much attention. i used the script that was present on this repo instead of the AM one. fixing it now and improving the overall logic of category and category page generation

[kazam0180]

done, we are down to 71 files changed, most of them are deleted files.

run by sync with AM worklfow:

am2pla-site (in this repo) now does only these things:

• creates apps.json (all apps, used on the Applications list)
• updates the stats in index.html

run by pages deployment workflow:

simple2json.py replaces the simple app info files in apps/ with json files (used by app.html)

gen_cats.py generates categories/*.json files

gen_cat_pages.py generates <category>.html files in root that render the category data from categories/

@andy5995 @ivan-hc

[kazam0180]

actually, apps.json too shouldn't be tracked in git, i will move that to the website deployment

[kazam0180]

done, now we're golden, apps.json is no longer tracked in git. it only lives on website, stats are also not updated in the repo

sync with AM now only removes unnecessary app info files and creates dumb info and icon files
when sync with AM commits new apps/removes apps, the website deployment workflow runs, it downloads x86_64-apps, stats-appimages and stats-portable.
it generates category json files, category html files, apps.json in root and updates the stats in index.html

[kazam0180]

there is a problem in website deployment, i am debugging it at https://github.com/Wolf-Pack-Clan/Portable-Linux-Apps.github.io

[kazam0180]

okay now everything seems good

[kazam0180]

@andy5995 @ivan-hc

[andy5995]

@kazam0180 50,000 changed lines is still way too many changes for a PR like this. And there are merge conflicts that would have to be fixed. Have you AI agent review the PR and the comments left by @ivan-hc and me, and post what it says here.

[kazam0180]

@kazam0180 50,000 changed lines is still way too many changes for a PR like this. And there are merge conflicts that would have to be fixed. Have you AI agent review the PR and the comments left by @ivan-hc and me, and post what it says here.

@andy5995 first of all i am not that stupid, i dont fully vibe code. i used AI to win time because it would take me like 3 or 4 days to get the html and css like this and AI is pretty good at UI and UX stuff.

all of the comments are already addressed since #167 (comment). i cant reduce any more changes. this is the minimum possible.
those 48k deletions are just the long category pages deleted (which are now no longer committed to the repository, generated and deployed on website only like i said before)

the conflicts are because of the deleted category pages. maybe i should re-add them 🤔

[andy5995]

@kazam0180 50,000 changed lines is still way too many changes for a PR like this. And there are merge conflicts that would have to be fixed. Have you AI agent review the PR and the comments left by @ivan-hc and me, and post what it says here.

@andy5995 first of all i am not that stupid, i dont fully vibe code. i used AI to win time because it would take me like 3 or 4 days to get the html and css like this and AI is pretty good at UI and UX stuff.

I didn't say you were stupid. I also use LLM to save time.

[kazam0180]

what i meant by "not that stupid" was that i can and have solved the issues in the PR myself

so you gonna take a look?
just see the new files and the modified workflow

[kazam0180]

@andy5995 should i create a new fork with the new files only? or discard commits then add new files