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

Fetch extra library info from NPM #158

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Fetch extra library info from NPM #158

wants to merge 6 commits into from

Conversation

davepagurek
Copy link
Collaborator

One of the bits of info we collected from library authors was NPM info so that we could do things like automatically create a CDN script tag, fetch the license if not manually provided, and show users how actively developed the library is. This does some of that initial fetching, currently in a quick-and-dirty little list of tags:

image

@Qianqianye This additional info isn't currently in the designs because we were considering it a nice-to-have that we might not have had the ability to collect, but it looks like it's going to be feasible. Design-wise, what do you think the best approach for something like this should be? One option is to try to mimic the style of tags the designers had in some old iterations for things like examples, which we later took out from that section. Another option is to just hide this info for now and think of a more permanent placement at a later date.

@limzykenneth
Copy link
Member

I generally like how ofxaddons does it with basically all data coming from GitHub and I think something like license can be a field of its own instead of being in the same place as the tags.

@davepagurek
Copy link
Collaborator Author

Update: the designers are OK with us using the tag design from this file
image

I've updated things slightly to look more in line with that, but also @stalgiag @outofambit we don't need this to hold up the rest of the site, so we're also OK just hiding this extra info for now if that makes things easier.

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