Skip to content

Comments

Implementation of audio logging.#350

Merged
kevindeland merged 1 commit intodevelopmentfrom
audio_logging
Aug 10, 2018
Merged

Implementation of audio logging.#350
kevindeland merged 1 commit intodevelopmentfrom
audio_logging

Conversation

@octavpo
Copy link
Collaborator

@octavpo octavpo commented Jul 10, 2018

No description provided.

Copy link
Collaborator

@kevindeland kevindeland left a comment

Choose a reason for hiding this comment

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

Awesome!

@kevindeland kevindeland merged commit e8ec416 into development Aug 10, 2018
@kevindeland
Copy link
Collaborator

Note that we will need to add a configuration setting to turn this off for the XPrize sites, so that we don't occupy our space on the FTP server with large audio files.

@kevindeland kevindeland deleted the audio_logging branch August 10, 2018 15:48
octavpo pushed a commit that referenced this pull request Aug 11, 2018
@JackMostow
Copy link
Contributor

@octavpo and @kevindeland - Can you make all config parameters default to the appropriate values for XPRIZE sites so that they don't need a config file?

octavpo pushed a commit that referenced this pull request Aug 11, 2018
@octavpo
Copy link
Collaborator Author

octavpo commented Aug 11, 2018

Note that we will need to add a configuration setting to turn this off for the XPrize sites, so that we don't occupy our space on the FTP server with large audio files.

I don't see the implementation of the .ini file on the development branch, so this will have to wait.

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.

3 participants