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

Rename contacts #133

Closed
wants to merge 26 commits into from
Closed

Rename contacts #133

wants to merge 26 commits into from

Conversation

burdges
Copy link
Contributor

@burdges burdges commented Oct 25, 2014

We finally have a non-hacky rename contacts feature for the GUI.

agl and others added 12 commits November 3, 2014 18:32
Does not resolve the existence of DELIVERY_SIGNATURE_INVALID messages in agl#134
… under TBB. There was an issue because PANDA runs over tor even in developer mode, but TBB uses port 9150 rather than the default tor port 9050.

Allow tor to be used by PANDA when running in developer mode

Oops
…ntacts branch. Renamed contactsSorted to lower case for consistency.
…eful for various planned features, including contat invitations, messages with multiple recipants, and editable notes fields for contacts.

In fact, I considered moving inputTextBlock to cli-inout.go, but it wouldn't save many imports in cli.go to do so.
…ating messages to specific recipiants and makes teh code more redable.
…sToEndOfQueue works as desired.

Revert "moveContactsMessagesToEndOfQueue should log whenever it runs.  Yey printf debuggin!"

This reverts commit aef4a3f.
@burdges burdges force-pushed the rename_contacts branch 8 times, most recently from 354bea3 to 40d1187 Compare November 3, 2014 21:30
@burdges
Copy link
Contributor Author

burdges commented Nov 4, 2014

This pull request is superseded by #137

@burdges burdges closed this Nov 4, 2014
@burdges burdges deleted the rename_contacts branch November 5, 2014 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants