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

Fix for #819 #1126

Closed
wants to merge 57 commits into from
Closed

Conversation

jankusanagi
Copy link
Contributor

A simple fix for #819. It can probably be made in a more elegant way, but this one seems to work well.

It just checks that the string in 'displayName' is not empty, because if it is, activities descriptions regarding the "null-named" object get broken.

Don't use displayName in activities descriptions if it's a 0-length
string
@jankusanagi jankusanagi changed the title Fix for https://github.com/e14n/pump.io/issues/819 Fix for #819 Oct 24, 2015
@strugee
Copy link
Member

strugee commented Feb 19, 2016

@jankusanagi can you rebase this on top of current master? This will allow Travis to run with fixed tests.

strugee and others added 11 commits February 22, 2016 13:04
For client javascript code written by E14N add stylized comments around license
notice. For 3rd party code verbatim copied from other projects create JavaScript
Web Labels table and keep scripts code unchanged.
We can not be sure that other inlined javascript code is free
Add missing { to a dictionary definition
only iterate over replies.items if it exists.
@jankusanagi
Copy link
Contributor Author

I think I messed this up :p
I'll close this PR and make a new one.

@jankusanagi jankusanagi closed this Mar 8, 2016
@jankusanagi jankusanagi deleted the fix-empty-object-names branch March 8, 2016 15:08
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.

8 participants