Commit graph

1090 commits

Author SHA1 Message Date
Ben Gotow 05714ee19f Merge pull request #52 from mbilker/fix-clean-script
Change atom to nylas in clean script
2015-10-09 10:23:28 -07:00
Evan Morikawa 044f5223ac Merge pull request #53 from mbilker/travis-update
Travis update
2015-10-09 10:20:37 -07:00
Evan Morikawa 69e5b2ac9a Add ChaosMonkey to test misbehaving servers
Summary:
`ChaosMonkey.unleashOnAPI()` will by default cause all API requests to 500

`ChaosMonkey.unleashOnAPI(timeoutMonkey: true)` will cause all API requests
to SOCKETTIMEOUT

`ChaosMonkey.unleashOnAPI(numMonkeys: 10)` will cause the next 10 API
requests to 500

`ChaosMonkey.unleashOnAPI(errorCode: 401, numMonkeys: 10)` will cause the
next 10 API requests to 401.

It must be manually invoked from the console on each window you want the
Monkeys wrecking havok.

It is available on the `window` object as well

This was created to manually test our server failure cases.

Test Plan: manual

Reviewers: drew, bengotow

Reviewed By: bengotow

Differential Revision: https://phab.nylas.com/D2133
2015-10-09 09:37:56 -07:00
Evan Morikawa a7aec14ca3 Fix contact ranking and add tests
Summary:
Contact ranking is now tested.

There was a bug whereby the RankingsJSONCache would only update in the
workerwindow. This regressed when Contact ranking moved exclusively into
the main window and separate composer windws requested rankings via ipc

Test Plan: New tests

Reviewers: drew, bengotow

Reviewed By: bengotow

Differential Revision: https://phab.nylas.com/D2134
2015-10-09 09:31:52 -07:00
Evan Morikawa eb0a915bc2 Fix scrollability in expanded activity sidebar
Summary:
If your screen is narrow or you are syncing a lot of stuff, the expanded
activity sidebar had no scroll affordance

Test Plan: manual

Reviewers: drew, bengotow

Reviewed By: bengotow

Differential Revision: https://phab.nylas.com/D2135
2015-10-09 09:31:37 -07:00
Ben Gotow 215fa0e4cb fix(initial-sync): When an error is encountered, do not start fetching from zero again
Summary:
Previously, when an error was encountered during initial mailbox sync we just started it
over after a retry delay. Recent API uptime issues mean that this was happening often and lots of
people were seeing sync retry many times. This is bad because the app is less performant while
it's syncing mail, and also generates unnecessary load as the app re-fetches threads it already has.

In this diff, there are new specs and functionality in nylas-sync-worker to start fetching
where we left off. This is typically going to be OK because the default sort ordering of the
threads endpoint is newest->oldest, so if new items have arrived since we started fetching
and page boundaries have changed, we'll get duplicate data rather than missing data. Connceting
to the streaming API as soon as we start the sync also ensures that we roll up any changes to
data we've already paginated over.

Test Plan: Run tests

Reviewers: drew, evan

Reviewed By: evan

Differential Revision: https://phab.nylas.com/D2132
2015-10-08 19:02:54 -07:00
Matt Bilker 7e8e3b6b97 fix(grunt): disable packaging under travis builds 2015-10-07 22:54:22 -04:00
Matt Bilker 67b0b5fd05 fix(travis): disable node 4 for now
throws errors everywhere regarding the compiler, nan, and node 4.1.2
2015-10-07 22:52:20 -04:00
Matt Bilker ca3d6d7e86 fix(travis): set compiler to clang 2015-10-07 22:36:28 -04:00
Matt Bilker 88be607b2c fix(build): disable codesign in travis builds 2015-10-07 22:35:51 -04:00
Matt Bilker cf5dc7dbbb fix(whitespace): coffeelint line 19 ends with whitespace 2015-10-07 22:32:28 -04:00
Matt Bilker a1b94d8239 fix(travis): another new github access token
found out you cannot use personal access tokens
github disables them if they are found in a commit
2015-10-07 22:26:45 -04:00
Matt Bilker e4367647ac fix(travis): update github access token 2015-10-07 22:15:10 -04:00
Matt Bilker d32d58d790 fix(travis): add github access key 2015-10-07 21:59:25 -04:00
Matt Bilker b968b41210 fix(grunt): change variable name for github access key 2015-10-07 21:57:45 -04:00
Matt Bilker 5b76da7d32 fix(travis): nvm does not like node 4 on osx 2015-10-07 21:56:03 -04:00
Matt Bilker b03d2fd52a fix(travis): add required APT packages 2015-10-07 21:52:42 -04:00
Matt Bilker af9dec17da fix(travis): switch to matrix build strategy 2015-10-07 21:51:30 -04:00
Matt Bilker c641c6fbbe fix(travis): accidentally removed the language selection 2015-10-07 21:46:14 -04:00
Matt Bilker cc07199c8f fix(travis): Update travis script to run on multiple oses 2015-10-07 21:42:48 -04:00
Matt Bilker 3bc1151b0f refactor(code): change atom to nylas in clean script 2015-10-07 21:34:07 -04:00
Evan Morikawa 1a265a9a5c Merge pull request #50 from lukegb/patch-1
fix(docs): correct SMPT->SMTP
2015-10-07 17:48:59 -07:00
Evan Morikawa b0e7769e94 fix(developer): visual improvements to developer bar
Summary: Make ETIMEDOUT red

Test Plan: manual

Reviewers: bengotow, dillon

Reviewed By: dillon

Differential Revision: https://phab.nylas.com/D2126
2015-10-07 17:28:45 -07:00
Evan Morikawa 55ff675f37 Update CONTRIBUTING.md and README.md to remove node requirement
No more warning message about needing Node v0.10. This got fixed :D

Adding more info about our commit guidelines.
2015-10-07 17:18:41 -07:00
Evan Morikawa 66e5371030 Upgrade to Electron 0.30.8. Remove unsused packages. Upgrade deps
- Upgrade node-sqlite3 from patched 3.0.2 to mainline 3.1.0

- Upgrade Electron to 0.30.8: NOTE: This is the latest in the 0.30 line.
  The 0.31+ line did a Chrome upgrade that we want to save for a later
  day. We've had a history of serious instabilities with these upgrades
  and are doing them conservatively

- Upgrade coffee-script from 1.9.0 to 1.10.0

- Package updates in /build folder
2015-10-07 16:52:49 -07:00
Drew Regitsky 09b91a0a28 feature(feedback): move feedback to a package, indicator for new msgs
Summary: Move all Intercom feedback code to a package. Change the appearance of the lower right question mark icon when a new intercom message is received (red, with repeating CSS bounce animation). New messages are detected by keeping the intercom window open (after the first time it's opened by the user), and listening for DOM mutations of particular classes.

Test Plan: manual

Reviewers: bengotow

Reviewed By: bengotow

Subscribers: evan

Differential Revision: https://phab.nylas.com/D2125
2015-10-07 13:55:54 -07:00
Luke GB 2de31790e9 fix(docs): correct SMPT->SMTP 2015-10-07 21:54:02 +01:00
dillon e72dc7464d fix(docs): add CLA to CONTRIBUTING.md
Test Plan: manual

Reviewers: evan, bengotow

Maniphest Tasks: T4193

Differential Revision: https://phab.nylas.com/D2128
2015-10-07 11:51:33 -07:00
Ben Gotow 7148c70e61 fix(send-availability): Make the package load into popout composers 2015-10-07 11:45:46 -07:00
Evan Morikawa fb264ef365 Merge pull request #49 from mbilker/node-v4
Compile on Node 4.x and 0.12
2015-10-07 10:55:51 -07:00
Evan Morikawa 86f76b9e94 Merge pull request #16 from sbstnmsch/feature-travis-ci
Add basic travis-ci integration
2015-10-07 10:55:23 -07:00
Dillon Forrest dd398e1d4f Merge pull request #30 from gardner/master
Adding node version information to build/README.md
2015-10-07 10:40:23 -07:00
Dillon Forrest 48cdf86f97 Merge pull request #31 from malonehedges/patch-1
Fix little typo
2015-10-07 10:38:57 -07:00
Dillon Forrest b94da7add7 Merge pull request #43 from boffbowsh/patch-1
Fix a couple of typos and a syntax error
2015-10-07 10:36:03 -07:00
Ben Gotow 14c5160a6d bump(version): 0.3.15 2015-10-07 10:19:16 -07:00
Matt Bilker 0edbf515fd Use npm's packaged node-gyp and combine configure and rebuild gyp steps 2015-10-06 23:41:19 -04:00
Matt Bilker a9f45b0776 Target node version used by APM 1.1.1 and Electron 0.30.7
node-gyp header layout change in node v4, causes package builds to fail.

This change forces node-gyp to download the correct version of node so
native extensions can be built.
2015-10-06 23:40:35 -04:00
Matt Bilker e78f081d21 Update build packages to allow building with node v4.1.2
coffee-lint-cjsx added to account for ignored peer dependencies in
npm 3+
2015-10-06 23:36:39 -04:00
Matt Bilker 1136319aea Update APM to 1.1.1
This is the latest version of APM at this time and is already in use
by the Atom editor.
2015-10-06 23:35:03 -04:00
Matt Bilker ecbb12ebef Change Electron version to 0.30.7 2015-10-06 23:34:02 -04:00
Brett Gerry 4bd8c37181 add more info on running against local sync engine
Test Plan: doc only change

Reviewers: evan, bengotow

Reviewed By: bengotow

Differential Revision: https://phab.nylas.com/D2119
2015-10-06 19:05:02 -07:00
Ben Gotow 5a2ddb67d5 fix(auth): Resize the auth window after an update to the token status 2015-10-06 17:58:07 -07:00
Ben Gotow 3ed1256fd1 fix(atom): Never allow the window to be set to fractional sizes 2015-10-06 15:32:48 -07:00
Ben Gotow a96760dbd3 fix(docs): Include a note about script/build in CONTRIBUTING.md 2015-10-06 14:31:00 -07:00
Ben Gotow b06ff30179 fix(docs): Add a note about our public Trello board 2015-10-06 14:28:16 -07:00
Paul Bowsher 51201fbc30 Fix a couple of typos and a syntax error 2015-10-06 22:26:33 +01:00
Ben Gotow 1ade3ea253 fix(docs): Add a note about building with Node 0.10.x 2015-10-06 11:51:31 -07:00
Malone Hedges 403dbd5d5c Fix little typo 2015-10-06 00:10:08 -07:00
Gardner Bickford 2ead4b0b6c Adding node version information to build/README.md 2015-10-05 23:43:33 -07:00
Ben Gotow 3896473e3b fix(attachments): When downloads fail, don't resolve to chained actions
Summary: Fixes Sentry 3319 and 3303

Test Plan: Run three new tests

Reviewers: dillon, evan

Reviewed By: dillon, evan

Differential Revision: https://phab.nylas.com/D2118
2015-10-05 17:57:24 -07:00