FYI - broken report export fixed

6 views
Skip to first unread message

Alon Salant

unread,
Oct 4, 2010, 1:51:36 PM10/4/10
to freehu...@googlegroups.com
I received a report this morning from Dugan at the MoBo Bicycle Co-op
in Cincinnati that report exports were failing by serving an empty
file.

This issue is now resolved.

The problem appeared to have been introduced by the last series of
upgrades (including a Rails upgrade) that we did when we moved to
EngineYard. This means that export has probably not been working since
end of June. We have test coverage for CSV export but it appears that
the Rails test infrastructure does not emulate the real implementation
sufficiently to product this issue in unit tests. Thanks to Dugan for
sending me the heads up!

Here's the commit with this fix:

http://github.com/asalant/freehub/commit/479ed4d8bafcc66bdb993c2ec86e312d2d23f009

Alon

Reply all
Reply to author
Forward
0 new messages