New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update cassandra.cql #118

Merged
merged 1 commit into from Sep 27, 2016

Conversation

Projects
None yet
3 participants
@inesromdhani
Contributor

inesromdhani commented Sep 23, 2016

fix the script, remove the ";" after declaring the PRIMARY KEY for each TABLE

Update cassandra.cql
fix the script, remove the ";" after declaring the PRIMARY KEY for each TABLE
@deruelle

This comment has been minimized.

Member

deruelle commented Sep 23, 2016

Thanks @inesromdhani for the contribution. Can you sign the Contributor License Agreement at http://www.telestax.com/open-source/#Contribute so we can accept the contributions and acknowledge you ?

@inesromdhani

This comment has been minimized.

Contributor

inesromdhani commented Sep 23, 2016

Done !!

@deruelle

This comment has been minimized.

Member

deruelle commented Sep 23, 2016

thanks @inesromdhani ! @vetss please review !

@vetss vetss merged commit 1bd0c30 into RestComm:master Sep 27, 2016

@deruelle deruelle removed the Peer Review label Sep 27, 2016

@vetss

This comment has been minimized.

Collaborator

vetss commented Sep 27, 2016

Hello @inesromdhani

  1. I merged your update, thanks for ypur work

  2. please remember that you do not need to run this script by yourself because SMSC GW creates all needed tables by it self. This script is only for your info.

@inesromdhani

This comment has been minimized.

Contributor

inesromdhani commented Sep 28, 2016

Thank you @vetss for your care, i am happy to trust my work

@deruelle

This comment has been minimized.

Member

deruelle commented Sep 28, 2016

Thanks @inesromdhani. Your contribution is acknowledged at https://telestax.com/acknowledgements/

@inesromdhani

This comment has been minimized.

Contributor

inesromdhani commented Sep 28, 2016

Thanks @deruelle

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment