Skip to content

Conversation

@01zulfi
Copy link
Contributor

@01zulfi 01zulfi commented Oct 28, 2025

Related #8526

icon: Copy.path,
onClick: async () => {
try {
await navigator.clipboard.writeText(notebook.id);
Copy link
Contributor

Choose a reason for hiding this comment

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

Use the writeToClipboard utility.

}
},
isHidden: !settingStore.get().isInboxEnabled,
multiSelect: false
Copy link
Contributor

Choose a reason for hiding this comment

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

No need for this.


---

### 3. Post data to Inbox API endpoint.
Copy link
Contributor

Choose a reason for hiding this comment

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

The json plus table seems redundant. A good example of how API documentation should look like is this: https://developer.paddle.com/api-reference/products/create-product

We should also add an example that can be run via curl. Another thing we need to add is how to send pre-encrypted data directly to the Notesnook server without going through the Inbox API server. Perhaps we should also add an example doing it using OpenSSL + cURL.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

A good example of how API documentation should look like is this: https://developer.paddle.com/api-reference/products/create-product

I've improved the current doc, not sure which specific changes you meant.

Another thing we need to add is how to send pre-encrypted data directly to the Notesnook server without going through the Inbox API server. Perhaps we should also add an example doing it using OpenSSL + cURL.

This would require us to explain how the data is encrypted, the docs for which are present in the other repo. Maybe we can say something like "You can skip the inbox server altogether and send encrypted data directly to the Notesnook server, learn more here (link to the inbox server repo)"

@01zulfi 01zulfi requested a review from thecodrr October 30, 2025 09: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.

2 participants