Jonathan Zacsh

should test files really be run once-per-@test? intentional? would patch be possible/acceptable?

3

**tl;dr** can I chnage bats, per the below note[1]? I was a bit confused that some failure-checking I placed atop a `.bats` file was executing once-per `@test`, as I expected...

malformed input vcard file causes unrecoverable error with no AIs for user

7

**tl;dr** user-input shouldn't cause crash, it should be reject (or a graceful way out should be offered, if it's not rejected). warning: I'm not sure where this issue should live...

techdebt: rm deprecated `IdempotentImportExecutorHelper` per #1107 per that PR, it's deprecated, but the details on fully cleaning up are not yet worked out. For now this is a placeholder TODO.

unlike #1104 I think this set of failures isn't actually my fault but is a real bug. the problem is there's a lack of test double being put in place...

cleanup of site and its maintenance/documentation

1

builds atop #1099 to try to cleanup our site's generation so updating is a bit easier to understand without making mistakes (like, eg, deleting the site).

for context, see the TODO on lines of `extensions/data-transfer/portability-data-transfer-microsoft/src/main/java/org/datatransferproject/transfer/microsoft/MicrosoftTransferExtension.java` in https://github.com/google/data-transfer-project/pull/1062/files inlined below: ```java // TODO(jzacsh,wmorland) don't keep adding here - just have the converters invoked automatically // when Media...

bug: missing test coverage for newer video logic in MediaContainerResource

1

see #1058 for context task: - [ ] add new test coverage for the video logic in MediaContainerResource

import fails with generic error - seems to be caused by uncaught exception in dialog

1

here's a copy/pasted stacktrace from my js console: stacktrace from TypeError in `ImportDialog.tsx:85` ``` ImportDialog.tsx:85 TypeError: Cannot read property 'parent' of null at e.addSubcomponent (ical.js:2610) at Object.updateTimezones (ical.js:94) at n.value...

issue#176: document `yarn add expo` for devs

1

(WIP) things I think would be helpful to cover: 1. [x] call out [`yarn add expo` that tasn explained](https://github.com/etesync/etesync-notes/issues/176#issuecomment-890487286) 1. [x] replace probably(?)-copy-pasta repo URL in the instructoins with etesync...

following README's dev instructions results fails (at least for me)

3

`yarn web` results in: ```sh yarn run v1.22.11 $ expo start --web Starting project at /tmp/src/etesync-notes Unable to find expo in this project - have you run yarn / npm...