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

Show file sizes on custom build page using GitHub API #106

Closed
localpcguy opened this issue Feb 6, 2013 · 5 comments
Closed

Show file sizes on custom build page using GitHub API #106

localpcguy opened this issue Feb 6, 2013 · 5 comments
Assignees

Comments

@localpcguy
Copy link

I was actually looking at the custom build page, if you don't already have this on track, I may look into adding the file sizes for each option (using the GitHub API to grab actual file sizes) and a running total size for the options selected. Shouldn't be super hard to do and would be valuable for the end user I think, as they get immediate feedback on whether it is worth removing an option or not.

@ghost ghost assigned gfranko Feb 6, 2013
@gfranko
Copy link
Owner

gfranko commented Feb 6, 2013

Great idea! I am using my other project, DownloadBuilder.js, for the custom build. All it is doing is pulling each minified file from Github and concatenating the files. Here is where I am making the jsonp request to Github within Downloadbuilder.js, so you could potentially save that data from the response, and pass it back to the user here and here. Let me know if that does not make sense.

@localpcguy
Copy link
Author

Ah, cool, that makes sense. I'll probably try to take a look at it in the next week or so.

@gfranko
Copy link
Owner

gfranko commented Feb 9, 2013

Or if you want to present the file size to a user before he or she downloads the files, then you will have to make separate Github requests. We can figure it out. Once SelectBoxIt v3.0 is done, then we can look at refactoring the custom build process.

@gfranko gfranko mentioned this issue Feb 12, 2013
15 tasks
@gfranko
Copy link
Owner

gfranko commented Feb 21, 2013

I didn't get around to adding this in the current release, so I'm going to close this issue. If you are interested in working on it in the future I will definitely reopen it.

@gfranko gfranko closed this as completed Feb 21, 2013
@localpcguy
Copy link
Author

Sounds good. I've been slammed with deliverables lately, sorry.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants