-
Notifications
You must be signed in to change notification settings - Fork 19
Rename library #4
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
Comments
I'd like to differentiate this library from the actual I'd propose looking for names that convey what it does. Something like the followings:
|
Good points! I like something like I think I've been throwing these types of questions to OpenAI and tune the outputs. It gave some good ideas for direction:
|
Yes, yes, yes! I've been thinking think for a while now, since I wrote https://github.com/readthedocs/reports/pull/20#discussion_r1042075502 when talking about the 2023 roadmap 💯
In fact, how much marketing-y we want to go? What about something like 🤩 |
Hah, I was also thinking of something goofy like that, I'm definitely not opposed! Let's ask the AI. |
Me and the AI went off the rails a bit, but it gave a few fun ideas:
🙃 There is definitely a product discussion in here though, and probably not something we need to consider just yet, but it is great to keep this in mind for later. We need to determine if we will be using injection of a group of features into documentation as a marketable feature, or is what we're marketing the individual features and injection is merely a technical implementation detail. I might lean a bit towards the latter right now, but I do think we'll need some language for the group of features that we are inject. |
readthedocs-embed-client or readthedocs-embed-client-js ? |
I think I like If called |
I'd prefer to not use things like
We should not have files for specific doctools, since the idea is to be agnostic regarding to that. The more I think about this, the more I like something like I think I will move forward with it unless you are oppose on that. |
I didn't think that If you get your superpowers from mate, then I like
I think it fits very well with what this library is? A home for all client-side stuff? Do you worry that functionality that isn't at home in the library would end up here? Because otherwise I think this is what it is: A lot of utility/helper functionality with no distinct theme. Some of it does UI, some of it does API calls, some of it is related to version logic, some of it is related to toggling the diff viewer.. lots of stuff!
It was a bad example to show that the naming convention looks nice if you want to have an extension, but no I agree that we will have a hard time maintaining multiple libraries, so it's nice to just have 1 library for everything 👍
|
Good input! I'm enjoying this exercise 😄 I think at very least it sounds like we all do not want to keep
Hah this is great point, lets scratch those.
Though I still do agree somewhat here too. In the end, it's a group of features that we're trying to lump together. I'd agree we can maybe avoid catchall sounding names though
The trouble with options like this is that we want to describe all of the features we're including in documentation, and the flyout is just one of those features that users might enable.
Documentation "superpowers", "powerups", "juice", "mate" are all great ideas to jump off from, but for the name of a product/feature these do probably feel too gimmicky. I'm still considering all these more from the product/marketing side foremost. For each of these we'd be telling users something like "Go to your project Settings, then Superpowers (Powerups/Mate/etc), and enable the flyout". This is where it starts to feel odd for me, they all feel out of place. However, I think what we're describing with these could be a great framing for a marketing page! It's not too over the top for a marketing page at all. It's maybe only a small upgrade from docs-tools, but I was thinking yesterday how "Doc addons"/ And I'd say that the marketing push could absolutely include some of the ideas that we've thrown around here. |
addons is a great name, and just to be clear, would you say then that the project and package would be called |
Great question, this is one of the remaining points I'm not yet settled on. I guess I was suggesting But that's just a guess too. Do others pull the same interpretation from just "Addons"? |
I would think that prefixing with |
Also not completely impossible to call it |
When we add HTTP proxy support for injecting HTML template snippets it could be called
I guess to elaborate:
So, RTD would be part of the name in most use cases still. But, also, I'm not strongly advocating here, just a note. |
I'm not so worried about having the |
I think we arrived at Addons for the name here -- I say we should probably rename this library here soon, so we can start using it properly! |
@agjohnson @ericholscher are we OK with these names? I can do the rename if so. |
Yeah, those names are good still I feel. If anyone has strong opinions otherwise, my main concern is mentioning |
Any reason not to just say |
Aye, I had the same thought above and don't feel strongly either way
That seems solid as well 👍 |
✔️
I created the
I also re-upload the js file as https://readthedocs-static-prod.s3.us-east-2.amazonaws.com/javascript/readthedocs-addons.js I think we have everything in place now. |
Awesome -- I'm glad we're using the same name in the code as in the marketing, that always helps make it easier to understand and communicate. 👍 |
@humitos I think this invite doesn't work for me because my NPM account is using my personal info. I keep getting failures accepting the invite. Could you send to NPM user agjohnson or [email protected]? |
@agjohnson done! |
Hrm, actually, it was the same effect: "Invitation was revoked" 🤷 Though, what is odd, is this is after it prompts me to accept the invitation into the RTD organization. It clear finds the invitation, and we know it's not revoked ... 🤔 I'll note to bug you about this again when I'm back, it's not really critical at all. In fact, I mostly think we'll just use the namespacing while internally working with the packages, but perhaps there will be a time we install everything through NPM |
FWIW, as an outside of group collaborator-ish person, I like this new name (and +1 on the naming consistency as well)! |
Before we start using this on user projects/publicly,
readthedocs-client
sounds like a CLI or API wrapper, and we've called this the doc embed JS so
far. There's probably wisdom is reusing that naming or something similar.
The text was updated successfully, but these errors were encountered: