I would prefer that we discuss significant changes like this before creating a PR.
Of course — I agree, and I’m happy to discuss this first.
I noticed this issue was already open: https://github.com/l5yth/potato-mesh/issues/481
, and thought it was worth tackling. After discussing it within our local community, it became clear that to onboard a large number of users (ingestors), we’ll likely need some form of user management — or perhaps something simpler like API key management.
I don’t have a strong opinion on the exact approach. My main goal is to enable onboarding as many ingestors as possible, with the aim of building a country-wide map view.
P.S. I’m aware this is a significant change, which is why I opened the PR as a draft 🙂
> I would prefer that we discuss significant changes like this before creating a PR.
Of course — I agree, and I’m happy to discuss this first.
I noticed this issue was already open: https://github.com/l5yth/potato-mesh/issues/481
, and thought it was worth tackling. After discussing it within our local community, it became clear that to onboard a large number of users (ingestors), we’ll likely need some form of user management — or perhaps something simpler like API key management.
I don’t have a strong opinion on the exact approach. My main goal is to enable onboarding as many ingestors as possible, with the aim of building a country-wide map view.
P.S. I’m aware this is a significant change, which is why I opened the PR as a draft 🙂
_Originally posted by @apo-mak in https://github.com/l5yth/potato-mesh/pull/543#issuecomment-3652383630_
<!-- github-sync-id:3733466072 -->
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Of course — I agree, and I’m happy to discuss this first.
I noticed this issue was already open: https://github.com/l5yth/potato-mesh/issues/481
, and thought it was worth tackling. After discussing it within our local community, it became clear that to onboard a large number of users (ingestors), we’ll likely need some form of user management — or perhaps something simpler like API key management.
I don’t have a strong opinion on the exact approach. My main goal is to enable onboarding as many ingestors as possible, with the aim of building a country-wide map view.
P.S. I’m aware this is a significant change, which is why I opened the PR as a draft 🙂
Originally posted by @apo-mak in https://github.com/l5yth/potato-mesh/pull/543#issuecomment-3652383630