Skip to content

feat: create calendar event by DialogFlow Fullfilment#111

Merged
pr-Mais merged 10 commits intoFirebaseExtended:@invertase/dialogflowfrom
pr-Mais:@invertase/dialogflow
Nov 30, 2022
Merged

feat: create calendar event by DialogFlow Fullfilment#111
pr-Mais merged 10 commits intoFirebaseExtended:@invertase/dialogflowfrom
pr-Mais:@invertase/dialogflow

Conversation

@pr-Mais
Copy link

@pr-Mais pr-Mais commented Nov 21, 2022

Welcome to the Experimental Extensions repo! If this PR adds a new extension,
thank you, and please make sure you follow the steps below! Otherwise, you can
disregard this message.

  • The new extension's id roughly follows the pattern
    [firebase-product]-[action]-[noun], like storage-resize-images
  • The following match the extension's id:
    • The folder that contains the extension (my-new-extension/)
    • The name field of my-new-extension/functions/package.json
    • The name field of my-new-extension/extension.yaml
    • A new entry in the packages array of the lerna.json file in the
      repository's root
  • I've added a generate-readme script to
    my-new-extension/functions/package.json that matches the other
    extensions in the repository
  • I've run the following npm commands at the root of the repository:
    • npm install
    • npm run generate-readmes
    • npm run format

@pr-Mais pr-Mais changed the title adding credentials to sessionClient and modify the the fullfilment webhook feat: create calendar event by DialogFlow Fullfilment Nov 25, 2022
const dialogflow = DialogFlow.v2beta1;
const sessionClient = new dialogflow.SessionsClient({
projectId: config.projectId,
...(fs.existsSync(config.servicePath) && { keyFilename: config.servicePath }),
Copy link
Member

Choose a reason for hiding this comment

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

This needs to be removed for the extension - might have to provide an auth client instance.


var dateTimeEnd = new Date(dateTime.getTime() + DEFAULT_DURATION * 60000);

var event = {
Copy link
Member

Choose a reason for hiding this comment

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

Is there a type for this?

Copy link
Author

Choose a reason for hiding this comment

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

not yet- will add it

@pr-Mais pr-Mais merged commit b21bbbf into FirebaseExtended:@invertase/dialogflow Nov 30, 2022
@pr-Mais pr-Mais deleted the @invertase/dialogflow branch November 30, 2022 12:40
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.

2 participants