Draft: feat: Add featured apps configura...

i did put in this PR for KDE discover. we'll see if it ends up being accepted. but if it does, thats one less thing to have to rely on yafti for so it can be streamlined
GitLab
Draft: feat: Add featured apps configurable URL (!933) · Merge requ...
Add function to check config for featured URL closes
27 Replies
jardon
jardonOP3mo ago
creating thread in case its helpful 🙂 @𝓯𝓻𝓮𝓪𝓴onomist sorry i should use the thread i created
𝓯𝓻𝓮𝓪𝓴onomist
yeah i eventually saw it i agree with your idea ill updoot
jardon
jardonOP3mo ago
requiring each downstream distro to have to build it themselves is not great
jardon
jardonOP3mo ago
seems rational and reasonable to me it seems like a weird thing to worry about modification of the featured list. i mean it doesnt have any impact on software sources ya know?
𝓯𝓻𝓮𝓪𝓴onomist
"i think youre failing to consider" is maybe a little accusative towards president of kde lmao i should chill a bit but yeah i think your idea is better
jardon
jardonOP3mo ago
ya the first part is the least chill. the two paragraphs after are at least really grounded in fact
𝓯𝓻𝓮𝓪𝓴onomist
yeah i posted
jardon
jardonOP3mo ago
i honestly didnt know he was the president 💀
𝓯𝓻𝓮𝓪𝓴onomist
i am a yapping machine
jardon
jardonOP3mo ago
thanks for weighing in. hopefully it helps
𝓯𝓻𝓮𝓪𝓴onomist
there are so many things that a user process could override easily in KDE like that's just how KConfig works readable plain text files if that's somehow a problem then why offer any settings at all :pepelaugh:
jardon
jardonOP3mo ago
the joys of collaboration
𝓯𝓻𝓮𝓪𝓴onomist
idk if theres a .kcfgc file u would need to add it to idk how discover does it looks like it does it in a weird way
jardon
jardonOP3mo ago
i wonder if hes worried more about the desktop vs mobile configs and it featuring very not optimized desktop apps on mobile
𝓯𝓻𝓮𝓪𝓴onomist
That is whoever sets the file's problem
jardon
jardonOP3mo ago
I don't disagree
𝓯𝓻𝓮𝓪𝓴onomist
@jardon well now your solution is getting code reviewed Lol
jardon
jardonOP3mo ago
i just saw that! t e a m w o r k you have any thoughts on his feedback? im really not sure why he's so worried about the featured list like messing with it is going to install malware @𝓯𝓻𝓮𝓪𝓴onomist
𝓯𝓻𝓮𝓪𝓴onomist
Yeah not a huge deal
jardon
jardonOP2mo ago
i still dont like the compromise, but maybe i should just take what i can get if it works then great its not like i even like kde
𝓯𝓻𝓮𝓪𝓴onomist
I like kde also nate the pull request beast himself agrees with your idea instead
jardon
jardonOP2mo ago
Do you have the ability to build and test my PR? I don't have a environment set up for it and the in-laws are in town
𝓯𝓻𝓮𝓪𝓴onomist
not right now @jardon let Mr president eat his dinner lmao It will get merged soon
jardon
jardonOP2mo ago
I'm tired of rebasing 🥺
𝓯𝓻𝓮𝓪𝓴onomist
He can do it That's what usually happens anyway
Want results from more Discord servers?
Add your server