bad logic on signin fixed