Kirill Shirinkin Fodoj

Fodoj commented on pull request elavretskiy/flashcards#26
@Fodoj

предлагаю обернут поиск по фликеру в отдельный сервисный класс FlickrService (app/services/flickr_service.rb)

Fodoj commented on pull request elavretskiy/flashcards#26
@Fodoj

почему не грузится автоматом из gemfile?

Fodoj commented on pull request elavretskiy/flashcards#26
@Fodoj

оступы всё ещё по два пробела, даже в js

Fodoj commented on pull request alx-t/flashcards#20
@Fodoj

я в гиттере уже предлагал вынести I18n.available_locales.map { |l| [I18n.t(l), l.to_s] } в хэлпер

Fodoj commented on pull request alx-t/flashcards#20
@Fodoj

я бы не стал так сокращать. Лучше полностью написать

Fodoj commented on pull request alx-t/flashcards#20
@Fodoj

можно поставить locale = перед всеми этими условиями ну или case when ещё =)

Fodoj commented on pull request alx-t/flashcards#20
@Fodoj

везде

Fodoj commented on pull request alx-t/flashcards#20
@Fodoj

дебаг убрать

@Fodoj
Do not require access keys
1 commit with 1 addition and 1 deletion
Fodoj opened pull request alienfast/middleman-aws#2
@Fodoj
Fix typo
1 commit with 1 addition and 1 deletion
Fodoj created branch fix-typo at Fodoj/middleman-aws
Fodoj opened pull request rumblelabs/asset_sync#301
@Fodoj
Add fog path_style option support
1 commit with 3 additions and 1 deletion
@Fodoj
  • @Fodoj 6f8333c
    Add path_style option support
Fodoj created repository mkdev-me/sinatra-api-example
@Fodoj
Fodoj commented on pull request alienfast/middleman-aws#1
@Fodoj

@rosskevin thanks!

@Fodoj
Fodoj commented on pull request alienfast/middleman-aws#1
@Fodoj

I've already reverted changes to readme

@Fodoj
Fodoj commented on pull request alienfast/middleman-aws#1
@Fodoj

Also, important part: step 3, if not on EC2, will raise a valid exception about unauthorised connection to AWS. So raising an exception for missing…

Fodoj commented on pull request alienfast/middleman-aws#1
@Fodoj

So, now: try to load from file if no file then warn user and try to load from env if nothing in env, then s3_sync and cloudfront gems will fallbac…

@Fodoj
@Fodoj