-
Notifications
You must be signed in to change notification settings - Fork 246
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
Readme update #88
Readme update #88
Conversation
cyfyifanchen
commented
Jul 11, 2024
- Consistency change
- restructure modals and extension part
- Adding 11labs as option in the config part
- Adding .next to gitignore
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the concept of interchangeable is not clear
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
It means it can be changed to the other extension. I think I do have a little paragraph describes it just right above the table. Another word choice is switchable, I ended up choose interchangeable, which I think is little bit better(I also asked GPT about it, he thinks the same). I am open to suggestions on any other better word choices. |
by far we do not yet have docs to describe how to switch extensions (you will need to manipulate manifest.json). the purpose of that section is to help developer understand the default set up, so emphasizing interchangeable is not very useful, and too early for a beginner. i would suggest we only describe the default setup here. if needed, you can add an additional section abt how to switch extensions. also i would recommend we do this after our graph designer is ready. |
Updated. |