codingteam / loglist

Reincarnation of the famous service
https://loglist.xyz/
MIT License
7 stars 4 forks source link

Make loglist buildable again #225

Closed Minoru closed 3 years ago

Minoru commented 3 years ago

Please see this commit's message to get an idea of what's going on here.

I tried to fix CI as well, but something's up with PostgreSQL on Travis. I suggest concentrating on testing with docker-compose for now, here's how:

$ cp devenv.example devenv.your_nickname
$ edit devenv.your_nickname   # Set recaptcha keys
$ source devenv.your_nickname

$ docker-compose up -d
# `loglist_db_1` comes from `docker-compose ps`
$ docker exec -it loglist_db_1 psql --username horta loglist
loglist=# insert into approver(name, email) values('dummy', 'dummy@example.com');

# Test at https://localhost:9000

# After submitting a couple quotes, get approval URLs for them:
loglist=# select 'http://localhost:9000/quote/approve?token=' || token from suggested_quote;
# Click the URLs this prints out.

loglist=# \q
$ docker-compose down

Fixes #223.

Minoru commented 3 years ago

@Orhideous, thank you for the review! I've processed all your feedback now, but GitHub won't let me re-request a review from you, so I'm asking for it verbally here Please take a look :)

Minoru commented 3 years ago

Squashed at @ForNeVeR's request.

ForNeVeR commented 3 years ago

A note regarding the CI: I'll take care of it from this moment.

ForNeVeR commented 3 years ago

Thanks a ton!

Minoru commented 3 years ago

Thanks for the review!