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

add neighbor seeding #18

Merged
merged 3 commits into from
Oct 18, 2014
Merged

add neighbor seeding #18

merged 3 commits into from
Oct 18, 2014

Conversation

nrn
Copy link
Member

@nrn nrn commented Oct 14, 2014

Add command line description for neighbors, and add a list of servers currently running on the farm to the neighbors list for all servers on the farm. This is an interesting idea I'd like to get everyone's input on, but it might not be the desired behavior.

This PR is not necessary for the other two to be merged and function as intended.

@paul90
Copy link
Member

paul90 commented Oct 15, 2014

An interesting idea. Removing the reliance on site entry point / navigation route as the sole criteria for determining which sites are in the current neighbourhood.

While a single static pre-defined neighbourhood, at server start, is interesting I think this is probably only a start.

I can't help but wonder if making this dynamic, and site specific, would be more interesting. Probably making it a specific 'page' and using the reference plug-in to define a site's seeded neighbourhood.

Making it a page, rather than defined at server start, makes it visible as something that can be shared, forked, and managed by a community.

@WardCunningham
Copy link
Member

This seems to address a bootstrap problem for site operators that want to host a farm for a specific community. Mike Caulfield comes to mind.

At one time I had the default Welcome Visitors page include a journal reference to fed.wiki.org so that the How To Wiki pages would be accessible. This became annoying "pollution" of the Recent Changes. This feature would pollute similarly but that might be the site operator's desired behavior.

I later placed a copy of How To Wiki in default pages where it could be found. This wouldn't be the most recent version, but twins would announce newer versions.

https://github.com/fedwiki/wiki-node-server/tree/master/default-data/pages

screen shot 2014-10-15 at 5 07 24 am

This proved adaquate for bootstraping documentation.

@michaelarthurcaulfield
Copy link

I've been having Students add a "Related Sites" portion to their Welcome page:

http://mits.wsuv.wiki:3000/view/welcome-visitors

In this case they can load "All Students" or just the students from their local group. The ability to only load subsets is important.

We have been calling these pages we load "circles", as in "Load your groups circle into your neighborhood".

This works great once it's running. But bootstrapping problems include:

  1. I have to construct this manually, which requires me getting URLs from students, etc.
  2. Students then have to get it from me, and requiring a cross-tab page drop in the first five minutes of using fedwiki is scary to some.
  3. Students have to fork the circle in order to make it useful, but it'd actually be more useful as a blue-haloed transclusion that gets auto-updated.

I'm not sure I fully understand the proposal -- would all sites on the server be seeded? If so that's too much.

But if I'm reading it correctly, the ability to seed a chosen site like sites.fed.wiki.org along with another wiki that provides a hand-crafted annotated directory would be awesome. I'd put both the group pages and the all student pages there, and then I'd just tell students what links to add under Related sites.

@nrn
Copy link
Member Author

nrn commented Oct 16, 2014

So the main thrust of the three pull requests, and the part that I really think needs to happen, is the ability to seed chosen sites using startup options. Seeding sites.fed.wiki.org is a great example, that and the root page of the current farm might be the two most useful.

This pr also has a bit that is pretty questionable, seeding every site currently running in a farm to every other site currently running in the farm. When I was working on the first part, I realized I could do the second part and wanted to float it out there as a possibility :)

@paul90
Copy link
Member

paul90 commented Oct 16, 2014

This pr also has a bit that is pretty questionable, seeding every site currently running in a farm to every other site currently running in the farm.

I also notice when testing, without specifying a neighborhood at start-up, that I am getting an undefined site appearing at the right of the list of sites.
screen shot 2014-10-16 at 11 59 28

When specifying a neighborhood at start-up, this problem does not appear.

@nrn
Copy link
Member Author

nrn commented Oct 16, 2014

Whoops, good catch @paul90, thought I tested that. Will correct it tonight.

Do we want to pull the automatic farm seeding magic out?

@WardCunningham
Copy link
Member

The farm seeding magic is interesting because it does something without further configuration. But I can see how the current magic might be annoying in some situations. Can we put it behind an option too, and maybe suggest that the magic might improve?

@nrn
Copy link
Member Author

nrn commented Oct 17, 2014

Sounds like a good plan.

@nrn
Copy link
Member Author

nrn commented Oct 17, 2014

That commit also fixes the unknown site bug from above.

paul90 added a commit that referenced this pull request Oct 18, 2014
@paul90 paul90 merged commit d126a8d into master Oct 18, 2014
@paul90
Copy link
Member

paul90 commented Oct 18, 2014

Looks good - some interesting comments from @michaelarthurcaulfield that will no doubt generate some future changes in this area.

@paul90 paul90 deleted the nrn/neighbors branch October 24, 2014 11:43
@paul90 paul90 mentioned this pull request Dec 14, 2014
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.

4 participants