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

Bitrot fixed #21

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Bitrot fixed #21

wants to merge 3 commits into from

Conversation

bitmonki
Copy link

@bitmonki bitmonki commented Sep 2, 2015

(Please disregard the previous pulll request, one of the md5.js files was re-added mistakenly)

Hallo,
Sofa is such an interesting little program, I just had to get it working! Thanks so much for sharing it.

Here are the changes I made, hope you find them useful. :)

I tried to make the changes as minimal as possible, and refrain from making 'improvements'. (I plan to do that in a separate project, now that its working, thinking of 'sofa-ng')

Chris (J) Hall, aka bitmonki
bitmonki@gmail.com

*** lists/index.js:
    Changed to return first element of key array rather than entire
    array.
*** views/lib/comments.js:
    Apparently, from within maps, CouchDB will only include files from
    the 'views/lib/' tree. Also changed to be more conformant to
    CommonJS require standards used elsewhere in the app. (This was the
    only macro invocation I could find in the entire project)
*** views/lib/md5.js:
    Added file so views/lib/comments.js could 'require' it.
*** views/post-page/map.js:
    Wasn't sorting properly, so nothing was being displayed, CouchDB
    reported 'getRow() is null'.
*** _attachments/script/md5.js:
    Added conditional 'export' statement
*** evently/profile/profileReady/selectors/#preview/click.js:
    Removed extra parameter to Mustache.escape() which was causing an
    empty string to be returned.
*** helpers/md5.js:
    Added 'export' statement
*** templates/edit.html:
    Correct post action
*** templates/post.html:
    Needed to be a 'div' for evently logic to use. Added script
    jquery.scrollTo.js
*** ERRATA.md
    Added.
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.

1 participant