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

Deprecate fontbakery.appspot.com with redirect #1406

Closed
chrissimpkins opened this issue Jun 28, 2017 · 13 comments
Closed

Deprecate fontbakery.appspot.com with redirect #1406

chrissimpkins opened this issue Jun 28, 2017 · 13 comments
Assignees
Labels

Comments

@chrissimpkins
Copy link
Member

chrissimpkins commented Jun 28, 2017

Observed behaviour

I tested the four v.2.020 Hack ttf typeface sets (regular, italic, bold, bold italic) on the Appspot application in Chrome 59.0.3071.115 and Firefox 55.0b2 (64-bit) on OS X 10.12.5 and receive test results for all of the sets except the italic set.

The problematic font file is properly detected when DnD on to the browser:

font_select

When section 3 is clicked, the initial popup that indicates that the analysis is taking place is displayed briefly and then it closes without results displayed in both browsers for the italic set. The application appears back at its default state like this:

default_app

Please let me know if I can provide additional information. Thanks!

Resources and exact process needed to replicate

https://github.com/chrissimpkins/Hack/releases/download/v2.020/Hack-v2_020-ttf.zip

@davelab6
Copy link
Contributor

@chrissimpkins the appspot page is stale. @felipesanches can you put up a redirect on that URL to the new one

@davelab6 davelab6 changed the title Appspot application error dependent upon the set of Hack typeface Deprecate fontbakery.appspot.com with redirect Jun 28, 2017
@chrissimpkins
Copy link
Member Author

thanks! will wait for latest release and give it another try

@davelab6
Copy link
Contributor

Oh, actually https://github.com/googlefonts/fontbakery/commits/master should be good fo ryou

@graphicore
Copy link
Collaborator

current, but not very mature: http://104.197.213.179/

Feel free to test and report issues here: https://github.com/googlefonts/fontbakery-dashboard/issues

@chrissimpkins
Copy link
Member Author

Oh, actually https://github.com/googlefonts/fontbakery/commits/master should be good fo ryou

Ok, thanks Dave.

current, but not very mature: http://104.197.213.179/

👍 thank you!

@chrissimpkins
Copy link
Member Author

@graphicore the application release in the URL that you linked fixed the issue with our italic build. The application works great. I like the stdout and stderr displays on the new page.

fontbakery

Will leave this open for Dave's request to include a URL redirect in #1406 (comment)

Thanks guys!

@chrissimpkins
Copy link
Member Author

chrissimpkins commented Jul 1, 2017

@davelab6 Is the master version (reported on pip install as v0.2.0) current with reported documentation on README? (Re-)Installed with pip from FontBakery Github repo (Ubuntu Linux 16.04.2) after identifying the following problem with:

pip install --upgrade git+https://github.com/googlefonts/fontbakery.git

Here is what I am seeing:

fontbakery_error

Help text differs from that on the README (doesn't include the fix-dsig subcommand and others) and I can't find an approach with the --autofix flag that allows for this command to execute properly. Let me know if you would like this as a formal issue report. I may have missed something on my end with the install...

@davelab6
Copy link
Contributor

davelab6 commented Jul 1, 2017 via email

felipesanches added a commit that referenced this issue Jul 3, 2017
This is important enough to justify a new release on Pypi. So, this commit also increases the version metadata to 0.2.1

(issue #1406)
@felipesanches
Copy link
Collaborator

@chrissimpkins I have just released fontbakery v0.2.1 with a fixed setup.py script including the script that was missing in the previous release. Can you try installing again and tell me if that works for you?

https://pypi.python.org/pypi/fontbakery/0.2.1

@chrissimpkins
Copy link
Member Author

Can confirm that I installed v0.2.1 from PyPI with pip install --upgrade fonttools, that the help menu is expanded with the subcommands that are included on the README file, and that fix-dsig, fix-nonhinting (only autofixes that I have tried so far...) both appear to execute without any issues. Thanks Felipe!!

m4rc1e pushed a commit to m4rc1e/fontbakery that referenced this issue Jul 4, 2017
This is important enough to justify a new release on Pypi. So, this commit also increases the version metadata to 0.2.1

(issue fonttools#1406)
m4rc1e pushed a commit to m4rc1e/fontbakery that referenced this issue Jul 4, 2017
This is important enough to justify a new release on Pypi. So, this commit also increases the version metadata to 0.2.1

(issue fonttools#1406)
@davelab6
Copy link
Contributor

davelab6 commented Jul 5, 2017 via email

@davelab6 davelab6 modified the milestone: 0.3.1 - Further improvements on reliability of checks and fixers Jul 24, 2017
@davelab6 davelab6 modified the milestones: 0.3.1 - Further improvements on reliability of checks and fixers, 0.3.0 Rewrite internals for modularity, speed Jul 25, 2017
@felipesanches
Copy link
Collaborator

We don't advertise the old webapp on README anymore. And the old app code has migrated into https://github.com/googlefonts/fontbakery-dashboard/

This issue does not belong here anymore.

@davelab6
Copy link
Contributor

davelab6 commented Aug 8, 2017 via email

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

No branches or pull requests

4 participants