Skip to content
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

Add notion_researcher extension #8070

Merged
merged 5 commits into from
Aug 29, 2023
Merged

Conversation

razgaon
Copy link
Contributor

@razgaon razgaon commented Aug 24, 2023

Description

This extension lets you add Arxiv papers to Notion instantly. The added page includes information about the paper (authors, date, subject). It also gathers the references from the papers, and explains the main topics using GPT 3.5.

Screencast

Add.Papers.mp4
Create.Database.mp4

Checklist

- Update README.md
- Update README.md
- Merge pull request raycast#8 from razgaon/production-preparation
- Fixed bug with semantic scholar and added toast.
- Merge pull request raycast#7 from razgaon/production-preparation
- Improved Semantic Scholar
- Merge pull request raycast#5 from razgaon/razgaon-readme-patch-1
- Merge pull request raycast#6 from razgaon/production-preparation
- Added automatic database creation.
- Update README.md
- Added image
- Added changelog and categories
- Updated icons
- Fixed lint issues
- Merge pull request raycast#4 from razgaon/feature/create-database
- Improved the UI and experience
- Added clear indication when adding papers
- Merge pull request raycast#3 from razgaon/feature/no-duplicates
- Checks if article already exists.
- Merge pull request raycast#2 from razgaon/feature/speed-enhancment
- Changed llm call to be concurrent with other calls.
- Update README.md
- Merge pull request raycast#1 from razgaon/feature/ml-explainability
- Added ml explaining
- Added support for multiple papers as input
@raycastbot raycastbot added the new extension Label for PRs with new extensions label Aug 24, 2023
@raycastbot
Copy link
Collaborator

Congratulation on your new Raycast extension! 🚀

We will review it shortly. Once the PR is approved and merged, the extension will be available on the Store.

@mil3na mil3na self-assigned this Aug 28, 2023
@mil3na
Copy link
Contributor

mil3na commented Aug 28, 2023

Hi!
Thank you for your contribution! My name is Milena and I will be reviewing your extension today.

First thing I noticed: after setting up the extension and calling the "Create Research Database" command, if my page doesn't have a name, I get the following error:
CleanShot 2023-08-28 at 16 58 38

Which is a bit confusing. Does the page really need to have a name?

On the preferences, why do you need a Readwise key? If it is not being used, please remove it.

I would make it more explicit on the README that you are pulling info from Arxiv. Maybe on the first paragraph? It would bring expectations closer to reality.

Finally, I love this extension and I wish I had access to it during my academic days :D Great job!

@razgaon
Copy link
Contributor Author

razgaon commented Aug 28, 2023

Hi Milena!
I appreciate you taking the time to review the extension!

  1. The Notion search API searches by title, so I don't think there is a workaround for this.
  2. The Readwise key is optional, but I added an explanation and another Optional warning.
  3. Will update the Readme right now to be more clear. Hoping to add more than just Arxiv soon!
    Is there anything else I need to do?

Thank you!

@mil3na
Copy link
Contributor

mil3na commented Aug 29, 2023

I was wondering if you could actually say that you need to have a title on your page for that error message.

And I think links to Readwise and Arxiv would make the description better.

After those small fixes, I guess we can merge your extension :)

@razgaon
Copy link
Contributor Author

razgaon commented Aug 29, 2023

I made changes based on your comments :)
I hope I understood what you meant correctly. Please let me know if there are any additional changes you'd like me to make!

Copy link
Contributor

@mil3na mil3na left a comment

Choose a reason for hiding this comment

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

Good enough! Thanks again for your contribution!

@mil3na mil3na merged commit 9119b47 into raycast:main Aug 29, 2023
@raycastbot
Copy link
Collaborator

Published to the Raycast Store:
https://raycast.com/raz_gaon/notion_researcher

@raycastbot
Copy link
Collaborator

🎉 🎉 🎉

Such a great contribution deserves a reward, but unfortunately we couldn't find your Raycast account based on your GitHub username.
Please link your GitHub account to your Raycast account to receive your credits and soon be able to exchange them for some swag.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new extension Label for PRs with new extensions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants