Skip to content
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

Show View -> Debug on OSX, ensure Debug Log is visible #2089

Merged
merged 2 commits into from
Mar 1, 2018

Conversation

scottnonnenberg
Copy link
Contributor

Two fixes to the import/registration workflow screens.

Copy link
Contributor

@gasi gasi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚢 Thanks for fixing this. I believe my fiancee ran into this. She got ‘error connecting to server’ and couldn’t send me the logs.

// eslint-disable-next-line no-param-reassign
template[3].submenu = [
template[windowIndex].submenu = [
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! 😄

Signal-Desktop/app/menu.js

Lines 279 to 281 in a3cf81b

const windowMenuTemplateIndex = includeSetup ? 4 : 3;
// eslint-disable-next-line no-param-reassign
template[windowMenuTemplateIndex].submenu = [

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep, this is a reimplementation, but it's ready for hotfix. :0)

@scottnonnenberg scottnonnenberg changed the base branch from development to master March 1, 2018 20:03
@scottnonnenberg scottnonnenberg merged commit c87a035 into master Mar 1, 2018
@scottnonnenberg scottnonnenberg deleted the flow-tweaks branch March 1, 2018 20:04
scottnonnenberg added a commit that referenced this pull request Mar 1, 2018
Fixed: In some cases on OSX, View menu would not have Debug Log option (#2089)

Fixed: In import/registration flow, choosing View -> Debug Log would do nothing (#2089)
scottnonnenberg added a commit that referenced this pull request Mar 1, 2018
Note: This release is equivalent to v1.5.2

Fixed: If interrupted in the middle of an import, next registration
would leave that imported data in the database (#2072)

Fixed: In some cases on OSX, View menu would not have Debug Log option
(#2089)

Fixed: In import/registration flow, choosing View -> Debug Log would do
nothing (#2089)

Fixed: In some cases contact/group syncs would turn off disappearing
messages in all conversations (#2086)

Fixed: On initial setup, conversations with disappearing messages
enabled would be at the top of the conversation list (#2084)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants