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

implement basic netlify functions:list command #211

Merged
merged 5 commits into from
Jul 8, 2019
Merged

Conversation

swyxio
Copy link
Contributor

@swyxio swyxio commented Jul 5, 2019

- Summary

its small, people want to check that their functions are properly formatted. theres more potential for local validation (functions doctor?) generally.

closes #209

pulling out the functions parsing to a separate file also helps us write other functionality.

image

- Description for the changelog

run ntl functions:list to see your locally correctly configured functions.

- A picture of a cute animal (not mandatory but encouraged)
image

@DavidWells
Copy link
Contributor

Looks good. We should probably add one more column here: Live (or whatever word)

The live column is true/false and specifies if the function is currenly live on the site or just local (undeployed) right now.

The live functions are included in the https://open-api.netlify.com/#/default/getSite endpoint

Example https://github.com/netlify-labs/oauth-example/blob/master/src/App.js#L155-L156

@swyxio
Copy link
Contributor Author

swyxio commented Jul 6, 2019

cool, TIL you added the Functions thing to your app

the word live i think collides with netlify dev --live. i'd go for deployed altho its a longer word.

i wonder if we can hash the function to understand if this specific version of the function has been deployed.

@swyxio
Copy link
Contributor Author

swyxio commented Jul 6, 2019

image
hmm, this swagger doesnt seem to have the available_functions that you us

@swyxio
Copy link
Contributor Author

swyxio commented Jul 6, 2019

added the deployment check
image

edited to flip the colors

image

@swyxio
Copy link
Contributor Author

swyxio commented Jul 6, 2019

reverted to table format. chalk seems to mess it up so i opted to remove chalk.
image

@DavidWells
Copy link
Contributor

nice! Looks good.

hmm, this swagger doesnt seem to have the available_functions that you us

Yes 😢. There is some serious syncing that needs to happen with API & the API spec.

@swyxio swyxio merged commit 5ef5230 into master Jul 8, 2019
@swyxio swyxio deleted the addfunctionslist branch July 8, 2019 03:47
@swyxio
Copy link
Contributor Author

swyxio commented Jul 8, 2019

There is some serious syncing that needs to happen with API & the API spec.

GRAPHQL ALL THE THINGS

swagger is so 2018

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.

Functions overview, more debugging info
2 participants