Skip to content
This repository was archived by the owner on Dec 15, 2022. It is now read-only.

Fix for Atom Issue #8794 Remove 'null' text from front-facing messages#57

Open
g23-aw wants to merge 3 commits intoatom:masterfrom
g23-aw:master
Open

Fix for Atom Issue #8794 Remove 'null' text from front-facing messages#57
g23-aw wants to merge 3 commits intoatom:masterfrom
g23-aw:master

Conversation

@g23-aw
Copy link

@g23-aw g23-aw commented Nov 27, 2015

Remove 'null' from alert messages if the item that is being printed is 'null'. Fix for atom issue atom/atom#8794.

This is the original message when the element is 'null':

messageoriginal

This is the message now:

message

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Extra space between for and path

@winstliu
Copy link
Contributor

I'm pretty sure this doesn't fix atom/atom#8794 since this only deals with Atom notifications that relate to Open on Github, not browser notifications.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants