okfn / handbook

Guides and introductions for participating in Labs and some of its projects.
http://okfnlabs.org/handbook/
170 stars 51 forks source link

Move "patterns" to top level directory as per our site architecture in #70 #79

Closed danfowler closed 8 years ago

danfowler commented 9 years ago

Patterns have now been converted and now must be re-organized under /pattern/ (i.e. some are still under /pattern/patterns/ and should be moved under /pattern/).

Glossary is special and go at site root i.e. /glossary.md (actual URL path will be /glossary/) (note glossary seems to be duped in two locations atm)

gsilvapt commented 8 years ago

@danfowler Should files be replaced, renamed or something else?

rufuspollock commented 8 years ago

@gsilvapt everything should move up to live under pattern. Can i suggest you post a short summary of what you plan to do here before you do it -- this is a generally recommended workflow.

e.g.

gsilvapt commented 8 years ago

In order to move the files under /patterns/patterns, some have the same name and you'll have a prompt asking if you want it to ignore those, rename or replace. My question was which one do you prefer to do in this case.

List of files with the same filename:

I can check the contents of each file to see if it is just a repetition or if the contents differ between files.

rufuspollock commented 8 years ago

@gsilvapt have you looked at the difference between these - are they just the same file. Also see #70 for site infor architecture. Please go ahead and do a first sample pull request showing what you plan to do.

gsilvapt commented 8 years ago

I was looking into all repeated files and:

Hope this helps! Sorry if I am not being too independent here but I prefer asking than doing things we will struggle to fix!

gsilvapt commented 8 years ago

The pull request 83 will go as follows:

Let me know what I can do to improve this and if this current structure solves this issue. Also, please have a look at the email print I sent with the PR just to ensure everything is correct and working.

gsilvapt commented 8 years ago

FIXED.