Closed lexicalunit closed 4 years ago
Merging #173 into master will not change coverage. The diff coverage is
100.00%
.
@@ Coverage Diff @@
## master #173 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 4 6 +2
Lines 214 265 +51
Branches 12 12
=========================================
+ Hits 214 265 +51
Impacted Files | Coverage Δ | |
---|---|---|
src/turbot/data.py | 100.00% <100.00%> (ø) |
|
...0279af35_keep_track_of_authorized_channels_per_.py | 100.00% <100.00%> (ø) |
|
...versions/9b107d322c46_use_bigint_for_author_ids.py | 100.00% <100.00%> (ø) |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 3b13fc0...e475ed5. Read the comment docs.
Added an !authorize
command so that different servers can authorized their own set of channels instead of only being able to set it on the command line.
Updated to resolve https://github.com/theastropath/turbot/issues/174.
When I started testing using PostgreSQL as the backend it became obvious quickly that some changes would need to be made to accommodate it.
psycopg2
, needed to be added.GROUP BY
query in sqlite doesn't work in a real database like PostgreSQL, so I re-wrote it in such a way that it will work on both sqlite and real databases.INTEGER
orBIGINT
for theauthor
columns. However in any real database you will needBIGINT
for these. So I added a migration to alter those columns. This doesn't actually affect the behavior of sqlite at all, but it makes things work on PostgreSQL.I also added a script and some documentation to help support future database migrations.
TODO: