-
Notifications
You must be signed in to change notification settings - Fork 179
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
add: support for multiselect in config form #296
base: master
Are you sure you want to change the base?
Conversation
it's funny that i have to review the PR of the guy that is trying to ruin my side business 😂 |
@jaruba are you the developer of RPDB? |
yeah, and incidentally also the dev lead at Stremio and maintainer of the addon sdk 😄 |
Ohh, my aim was to just make the svg manupilation logic open source and configurable. Rating fetching part is very hard to keep consistent. People will pay you just for the rating part ig. |
while i can't do much about it, i can at least share my POV: i invented this concept many years ago in an attempt to expand the ratings possibilities of apps, many have made variations of it since, for specific use cases, great ideas are meant to be spread i guess; imo building a self-hosted solution for this for Stremio users is cool, i can't presume ill intent if the purpose is to offer something for free to people, but then you started hosting on elfhosted, and if elfhosted will end up asking for money for this from users (which is very likely, as elfhosted is a for-profit company), then aren't you just building the same thing all over again so someone else can try to make money from it? which is not cool and not in line with "free alternative" mindset 🤷♂️ |
I didn't host it anywhere apart from the ngrok link that I have shared in the repo itself. That is for my personal use and has high downtimes. Someone took the code and hosted it. I have no affliation with them. I asked them to host it just to provide POC to people who don't have access/ don't know how to host for themself. I have no intentions on monetising it. |
Not merging it? |
@hexdecimal16 srry about that, i never found time for it and it was a low priority task, my fear with this PR is that it complicates the settings quite a bit, if we choose to move addon settings to the apps (instead of the browser) at some point, then the multiselect option becomes hard to implement in all the different clients |
this will add option to use mutli select form field when setting up configuration for manifest.json