search

thervh70 / ContextProject_RDD

1 stars 0 forks source link

Update URL to new Octopeer server url #193

Closed mpsijm closed 8 years ago

mpsijm commented 8 years ago

Since @aaronang has the new Octopeer server running, we can change the URL in our system as well :)

coveralls commented 8 years ago

Coverage Status

Coverage remained the same at 91.724% when pulling 15350d0bf85b6b4d6c2ddb7f6cc5ca0cb9ff19c6 on update-ocopeer-url into e9e8326e9e480bd7119b2c4ae25b08aa35cae969 on dev.

thervh70 commented 8 years ago

500 errors experience by @mpsijm ...

coveralls commented 8 years ago

Coverage Status

Coverage increased (+0.07%) to 91.826% when pulling 507e02b26a45925eed32530300876655ef44abe6 on update-ocopeer-url into 7556020e6e1ee026a89fabde39ced5a062890806 on dev.

mdingjan commented 8 years ago

Works for me. Ready for merging!

mpsijm commented 8 years ago

The 500s were caused by a duplicate user on the server (for which Mitchell didn't have a problem luckily). Robin and I both had our username duplicate in the database. This is probably due to concurrency issues on the server side.

thervh70 commented 8 years ago

So how will we prevent this from happening in the future? Also: how will we (@mpsijm and I) get a working version of the extension when our names are dulpicated..

mpsijm commented 8 years ago

Unfortunately, we can't do anything with the database without @aaronang. He can either flush the database (this doesn't prevent from duplicates happening in the future) or try to fix it (which takes more time).

mpsijm commented 8 years ago

Okay, @aaronang flushed the database and it works for me now :) Can you guys please also take another look? :)

mdingjan commented 8 years ago

I've had another look; it still works nicely for me. :)

thervh70 commented 8 years ago

Nope Nope.

thervh70 commented 8 years ago

Aaron made the username unique. Therefore I would like @mpsijm and @mdingjan to test this one last time. Then one of you can merge this!

mpsijm commented 8 years ago

I'm lonely again, in the database that is :P Works for me, so if anybody else can confirm it works, it can be merged :)

mdingjan commented 8 years ago

Still works for me! Will merge!