Skip to content
This repository has been archived by the owner on Jul 31, 2019. It is now read-only.

[S]Fix return with no results #16

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

Conversation

Xhark-Blues
Copy link
Contributor

No description provided.

Copy link
Contributor

@Istar-Eldritch Istar-Eldritch left a comment

Choose a reason for hiding this comment

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

What is the muninn.status file?

src/service.ts Outdated
@@ -69,6 +73,7 @@ export default async function init({
const property = payload.properties.attributes[zset];
if( property ) {
return all(property.map((value: string) => {
console.log(`${value} inserted as ${zset}`);
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this log required here?

Copy link
Contributor

@Istar-Eldritch Istar-Eldritch left a comment

Choose a reason for hiding this comment

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

Is the comment really required?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants