Skip to content

feat: add Cubox integration for saving entries #3385

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

Merged
merged 4 commits into from
Apr 8, 2025
Merged

Conversation

LionChenA
Copy link
Contributor

@LionChenA LionChenA commented Apr 1, 2025

close #1070 .
Added the feat on existing code, with i18n for English and Chinese.

  • Add "Save to Cubox" feature
  • Implement Cubox API integration
  • Support Cubox API key configuration in settings
  • Add localization for Cubox-related features

Questions:

  • Cubox API requires Pro—would appreciate someone testing it.
  • Not sure about the SVG—it’s AI-generated for now.

CleanShot 2025-04-01 at 20 21 00@2x

@follow-reviewer-bot
Copy link

Thank you for your contribution. We will review it promptly.

Copy link

vercel bot commented Apr 1, 2025

@LionChenA is attempting to deploy a commit to the RSS3 Team on Vercel.

A member of the Team first needs to authorize it.

@follow-reviewer-bot
Copy link

Suggested PR Title:

feat: add Cubox integration for saving entries

Change Summary:
This PR introduces integration with Cubox, allowing users to save entries directly to their Cubox account. Key changes include the addition of new settings for enabling Cubox, configuring an API token, and implementing the backend command to handle the save operation. Several UI components were also updated to accommodate this integration, including localization updates for English and Chinese.

Code Review:
No change requests necessary.

@LionChenA LionChenA changed the title feat(integration): add Cubox support feat: add Cubox integration for saving entries Apr 2, 2025
@hyoban
Copy link
Member

hyoban commented Apr 2, 2025

I see that Cubox has two APIs. Do you think adding an option to let users choose whether to save the web page or the content is necessary?

@LionChenA
Copy link
Contributor Author

I see that Cubox has two APIs. Do you think adding an option to let users choose whether to save the web page or the content is necessary?

Do you mean the url and memo types? Yes, I do notice the memo option.

The memo type allows users to save a specific part of an article to Cubox, whereas the url type saves the full webpage. However, considering Folo’s mission of gathering resources from RSS and incorporating preprocessing workflows, I see memo as more aligned with fleeting notes or advanced processing, such as annotations, which can later be refined in Cubox or even more systematically in Zotero.

Of course, this is just my perspective. Folo provides an environment for quickly browsing and filtering valuable articles. If an article proves valuable, I prefer saving it to Zotero to retain the full context.

Since I don’t use Cubox in my daily workflow, I’d suggest letting users decide whether to add this option. Alternatively, I can implement it first and gather user feedback to see if it meets their needs.

- Add Cubox integration with API token support
- Add enableCuboxAutoMemo setting for auto memo mode
- Add i18n translations for Cubox settings
- Support saving selected text as memo when auto memo mode is enabled
@LionChenA
Copy link
Contributor Author

I see that Cubox has two APIs. Do you think adding an option to let users choose whether to save the web page or the content is necessary?

I've added calls for both APIs. When the memo mode is enabled, clicking the save button will check whether any text is selected — if so, it uses the memo API; otherwise, it falls back to the url API.

As for the SVG of Cubox, I’m currently in touch with their team. For now, it’s using the Eagle icon.

Lastly, here’s a preview.

CleanShot 2025-04-07 at 12 56 50

@hyoban
Copy link
Member

hyoban commented Apr 8, 2025

Thank you very much for what you did, I really like the idea of ​​auto memo mode.

Regarding the icon, I created a simplified version. If the cubox team provides an official icon, I will be happy to replace it, please feel free to create another PR to update it.

CleanShot 2025-04-08 at 17 21 09@2x

@hyoban hyoban merged commit 02c59ca into RSSNext:dev Apr 8, 2025
4 of 6 checks passed
@follow-reviewer-bot
Copy link

Thank you for your contribution! 🎉

Your pull request has been merged and we really appreciate your help in making this project better. We hope to see more contributions from you in the future! 💪

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.

Add cubox support in the integration
2 participants