partially implement #579 reduce amount of data loaded #580

syjer · 2019-01-13T20:26:01Z

Note: this work will proceed on master directly.

… where possible

… import

coveralls · 2019-01-13T20:33:54Z

Coverage increased (+0.01%) to 53.512% when pulling 3a1d869 on 579-reduce-amount-of-data-loaded into c6d78f1 on master.

cbellone

👍

syjer added 9 commits January 13, 2019 16:18

#579 initial work, try to replace Event with EventAndOrganizationId

fb38eb6

#579 continue to narrow down the Event type to EventAndOrganizationId…

0f0e773

… where possible

#579 don't load the full event when sending the email

0c92571

#579 don't load the full event in ConfigurationManager, remove unused…

11057da

… import

#579 narrow down the type from Event to EventAndOrganizationId

28016b6

#579 don't load the full SpecialPrice model

f296ddc

#579 I18nManager -> load the locale integer from event directly

29abb38

#579 use EventAndOrganizationId type where possible

ee528ae

#579 use EventAndOrganizationId type where possible

3a1d869

syjer requested a review from cbellone January 13, 2019 20:26

cbellone approved these changes Jan 14, 2019

View reviewed changes

cbellone merged commit f5f6233 into master Jan 14, 2019

syjer deleted the 579-reduce-amount-of-data-loaded branch June 27, 2019 09:38

Provide feedback