issue: https://linear.app/comm/issue/ENG-4681/create-action-objects-for-all-actions
first two actions will be updated in next diff
Details
Details
Tested that the actions work as expected
Diff Detail
Diff Detail
- Repository
- rCOMM Comm
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
| lib/actions/thread-actions.js | ||
|---|---|---|
| 161 ↗ | (On Diff #31685) | In the future we will need to add a way to pick the keyserver we want to create a new community on. DMs and thick threads will also be handled differently. |
| lib/actions/message-actions.js | ||
|---|---|---|
| 484 ↗ | (On Diff #31685) | We shouldn't hide await inside of an another expression |
Comment Actions
This diff introduced a bunch of imports that don't include the .js file suffix. We aren't able to enforce that suffix via ESLint right now, but we should still try to always include the .js file suffix
Comment Actions
This diff introduced a bunch of imports that don't include the .js file suffix. We aren't able to enforce that suffix via ESLint right now, but we should still try to always include the .js file suffix
Sorry about this. I noticed that VSCode sometimes even removes the .js that were previously present. I'm trying to add them even if that doesn't cause any errors