Adding Modules... Please Please advise

A Former User

A Former User

/// as used instead of ``` as they crash my system (the modules) just blank screen, will be removed when the issue is resolved.

bhepler

@wjdw87 Why do you have the newsfeed module specified twice? You have:

{
	module: 'newsfeed',
	position: 'bottom_bar',
	config: {
		feeds: [
			{
				title: "Bloomberg Baby!",
				url: "https://www.bloomberg.com/feeds/podcasts/etf_report.xml"
			}
		],
		showSourceTitle: true,
		showPublishDate: true
	}
},
{
	module: 'newsfeed',
	position: 'bottom_bar',
	config: {
		feeds: [
			{
				tittle: "BBC Sci-Tech",
				url: "http://feeds.bbci.co.uk/news/video_and_audio/technology/rss.xml?edition=uk"
			}
		],
		showSourceTitle: true,
		showPublishDate: true
	}
},

Would it not make more sense to just put both feeds in the feed array?

{
    module: 'newsfeed',
    position: 'bottom_bar',
    config: {
        feeds: [
            {
                title: "Bloomberg Baby!",
                url: "https://www.bloomberg.com/feeds/podcasts/etf_report.xml"
            },
            {
                title: "BBC Sci-Tech",
		url: "http://feeds.bbci.co.uk/news/video_and_audio/technology/rss.xml?edition=uk"
            }
        ],
        showSourceTitle: true,
        showPublishDate: true
    }
}

(Also, you misspelled “title” in your second newsfeed. You have an extra “t”.)

A Former User

@bhepler said in Adding Modules... Please Please advise:

@wjdw87 Why do you have the newsfeed module specified twice? You have:

{
	module: 'newsfeed',
	position: 'bottom_bar',
	config: {
		feeds: [
			{
				title: "Bloomberg Baby!",
				url: "https://www.bloomberg.com/feeds/podcasts/etf_report.xml"
			}
		],
		showSourceTitle: true,
		showPublishDate: true
	}
},
{
	module: 'newsfeed',
	position: 'bottom_bar',
	config: {
		feeds: [
			{
				tittle: "BBC Sci-Tech",
				url: "http://feeds.bbci.co.uk/news/video_and_audio/technology/rss.xml?edition=uk"
			}
		],
		showSourceTitle: true,
		showPublishDate: true
	}
},

Would it not make more sense to just put both feeds in the feed array?

{
    module: 'newsfeed',
    position: 'bottom_bar',
    config: {
        feeds: [
            {
                title: "Bloomberg Baby!",
                url: "https://www.bloomberg.com/feeds/podcasts/etf_report.xml"
            },
            {
                title: "BBC Sci-Tech",
		url: "http://feeds.bbci.co.uk/news/video_and_audio/technology/rss.xml?edition=uk"
            }
        ],
        showSourceTitle: true,
        showPublishDate: true
    }
}

(Also, you misspelled “title” in your second newsfeed. You have an extra “t”.)

you can add them to the same array?

bhepler

@wjdw87 Indeed, you can. The documentation for the newsfeed module gives a simple example.

A Former User

Ah cool, updated that, but still have issues around the modules i’ve /// out

much appreciated though!

bhepler

@wjdw87 Okay, I finally had some time so I looked at the file you linked.

As @strawberry-3-141 said, you have an issue on line 100, part of the configuration of the camera module. You have: config: selfieInterval: 3, // Time interval in seconds before the photo will be taken.

You need to follow the config key with a JSON object, designated by curly braces. So the camera module section should look like so:

{
	module: 'camera',
	position: 'top_center',
	config: {
		selfieInterval: 3    
		emailConfig: {
			service: 'gmail', 
			auth: {
				user: '<xxx@xxx.com>',
				pass: '<xxx>' 
				},
			},
		}
	},

It doesn’t help that the config section is wrong in the README.MD in the camera module. I’ll see if I can’t get a PR to fix that.

bhepler

Additionally, you are missing a closed square bracket in your configuration for the VoiceControl module. After you define your keywords, you need to close the array. The second to last line of your voicecontrol configuration is where it needs to go. Like so:

{
	module: 'voicecontrol',
	position: 'bottom_left',
	config: {
	models: [
		{
			keyword: "Show Camera",
			description: "Say 'Show Camera' to display camera",
			file: "showCamera.pmdl",
			message: "SHOW_CAMERA"
		},
		{
			keyword: "Hide Camera",
			description: "Say 'Hide Camera' to hide camera",
			file: "hideCamera.pmdl",
			message: "HIDE_CAMERA"
		},
		{
			keyword: "Selfie",
			description: "Say 'Selfie' when camera is visible",
			file: "selfie.pmdl",
			message: "SELFIE"
		}
	]  // **HERE**
},

A Former User

@bhepler said in Adding Modules... Please Please advise:

@wjdw87 Okay, I finally had some time so I looked at the file you linked.

As @strawberry-3-141 said, you have an issue on line 100, part of the configuration of the camera module. You have: config: selfieInterval: 3, // Time interval in seconds before the photo will be taken.

You need to follow the config key with a JSON object, designated by curly braces. So the camera module section should look like so:
{
	module: 'camera',
	position: 'top_center',
	config: {
		selfieInterval: 3    
		emailConfig: {
			service: 'gmail', 
			auth: {
				user: '<xxx@xxx.com>',
				pass: '<xxx>' 
				},
			},
		}
	},
It doesn’t help that the config section is wrong in the README.MD in the camera module. I’ll see if I can’t get a PR to fix that.

@bhepler said in Adding Modules... Please Please advise:

Additionally, you are missing a closed square bracket in your configuration for the VoiceControl module. After you define your keywords, you need to close the array. The second to last line of your voicecontrol configuration is where it needs to go. Like so:
{
	module: 'voicecontrol',
	position: 'bottom_left',
	config: {
	models: [
		{
			keyword: "Show Camera",
			description: "Say 'Show Camera' to display camera",
			file: "showCamera.pmdl",
			message: "SHOW_CAMERA"
		},
		{
			keyword: "Hide Camera",
			description: "Say 'Hide Camera' to hide camera",
			file: "hideCamera.pmdl",
			message: "HIDE_CAMERA"
		},
		{
			keyword: "Selfie",
			description: "Say 'Selfie' when camera is visible",
			file: "selfie.pmdl",
			message: "SELFIE"
		}
	]  // **HERE**
},

Many thanks