Skip to content

feat(playground): added more playground examples#524

Open
YonatanKra wants to merge 1 commit into
CatchTheTornado:masterfrom
YonatanKra:more-playgrund-examples
Open

feat(playground): added more playground examples#524
YonatanKra wants to merge 1 commit into
CatchTheTornado:masterfrom
YonatanKra:more-playgrund-examples

Conversation

@YonatanKra
Copy link
Copy Markdown
Collaborator

Closes #181

Comment on lines +117 to +123
return {
data: dataSet['date'],
dailyconfirmed: dataSet['dailyconfirmed'],
dailydeceased: dataSet['dailydeceased'],
dailyrecovered: dataSet['dailyrecovered']
}
})
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

let's fix the indentation

Copy link
Copy Markdown
Contributor

@mhagmajer mhagmajer Oct 28, 2020

Choose a reason for hiding this comment

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

Also return is not needed here

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This mapping is essentially picking keys from objects. Shall we change key names to make this a little more exciting?

return "Hi, this is AskQL server! It's " + new Date().toString();
},
}),
statewise: resource({
Copy link
Copy Markdown
Contributor

@mhagmajer mhagmajer Oct 28, 2020

Choose a reason for hiding this comment

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

Let's make this custom resource name more specific

Copy link
Copy Markdown
Contributor

@mhagmajer mhagmajer left a comment

Choose a reason for hiding this comment

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

Thanks for adding more examples @YonatanKra.

All of those are currently using fetch and map maybe we can use add filter to some of them to make them more interesting?

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.

Add more examples to the AskScript Playground

2 participants