Skip to content
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

WIP: Local only communities #3889

Closed
wants to merge 3 commits into from
Closed

WIP: Local only communities #3889

wants to merge 3 commits into from

Conversation

wont-work
Copy link
Contributor

Adds a new flag to communities to mark them as "local only".

Unlike other fediverse software these communities will still federate, but posts will be effectively locked to those who are not in the same instance as the community.

This is an untested proof of concept hacked together in a few hours. I'm sending it just to get feedback on my overall approach. (And of course asking if upstream even wants this functionality in the first place)

Use lemmy-ui from https://github.com/ShittyKopper/lemmy-ui/tree/feat/local_only_communities, requires TS binding regeneration.

Adds a new flag to communities to mark them as "local only".
Unlike other fediverse software these communities will still federate,
but posts will be effectively locked to those who are not in the same
instance as the community.
@wont-work
Copy link
Contributor Author

(you can tell i didn't test shit because i just caught that there is nothing preventing remote commenting in the apub layer, working on it)

@wont-work wont-work closed this Aug 31, 2023
@Nutomic
Copy link
Member

Nutomic commented Sep 1, 2023

Im actually planning to work on this myself soon as part of NLnet funding. Its also a very complicated change as it touches most parts of the code, so I suggest you pick something easier to start contributing.

@wont-work
Copy link
Contributor Author

Its also a very complicated change as it touches most parts of the code, so I suggest you pick something easier to start contributing.

yeah i kinda figured that out the hard way 😰

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants