Hello,

At the config I get an error when trying to import Fontdeck: Home » Administration » Configuration » User interface

Error msg:
There was an error importing font information for project http://fontdeck.com/project/6390 on this domain (ABC.com) from Fontdeck.

Has anyone else had this issue and found a way to resolve it?

Thanks

CommentFileSizeAuthor
#8 fontdeck-number-input-1133688-7.patch802 bytesAnonymous (not verified)
#6 fontyourface-1133688-6.patch794 bytesAnonymous (not verified)

Comments

sreynen’s picture

sreynen commented
Title: Fontdeck error on import » Improve Fontdeck project number input
Version: 7.x-1.2 » 7.x-1.x-dev
Component: Miscellaneous » User interface
Category: bug » task
Priority: Normal » Minor

We talked about this in IRC, but in case anyone else is running into it: only the project number, e.g. "6390" should be entered into the field, not the whole URL. rob_dean suggested adding the URL as a prefix to the field to make the interface a little clearer, which I think is a good idea.

rovo’s picture

rovo
Location Delaware
commented
Title: Improve Fontdeck project number input » Fontdeck error on import
Version: 7.x-1.x-dev » 7.x-1.2
Component: User interface » Miscellaneous
Category: task » bug
Priority: Minor » Normal
Status: Active » Fixed

I misunderstood the directions underneath the field
"This is the number in the URL http://fontdeck.com/project/####"

Was trying to put my entire URL into the field, when I was to only put the last numerical part, represented by the # signs.

Thank you.

sreynen’s picture

sreynen commented
Title: Fontdeck error on import » Improve Fontdeck project number input
Version: 7.x-1.2 » 7.x-1.x-dev
Component: Miscellaneous » User interface
Category: bug » task
Priority: Normal » Minor
Status: Fixed » Active

Looks like we cross-posted. I'm leaving this open to remind me to improve the UI

sreynen’s picture

sreynen commented
Issue tags: +Novice

Tagging.

sreynen’s picture

sreynen commented
Version: 7.x-1.x-dev » 7.x-2.x-dev

This should happen in 7.x-2.x now.

Anonymous’s picture

Anonymous (not verified) commented
Status: Active » Needs review
StatusFileSize
new fontyourface-1133688-6.patch794 bytes

This is my first attempt at submitting a patch, so please advice me if I did something wrong.

Added field prefix "http://fontdeck.com/project/" shortened the field to 10 characters.
Changed the description to "Only paste the project number part in the URL."

Thank you.

Status: Needs review » Needs work

The last submitted patch, fontyourface-1133688-6.patch, failed testing.

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs work » Needs review
StatusFileSize
new fontdeck-number-input-1133688-7.patch802 bytes

And here is the second attempt.

Added field prefix "http://fontdeck.com/project/" shortened the field to 10 characters.
Changed the description to "Only paste the project number part in the URL."

Thank you.

sreynen’s picture

sreynen commented
Status: Needs review » Reviewed & tested by the community

This looks good. I'll try to get it committed soon.

BarisW’s picture

BarisW commented
Status: Reviewed & tested by the community » Fixed

Committed, thanks!

See: http://drupal.org/user/1327098/track/code

Automatically closed -- issue fixed for 2 weeks with no activity.