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

Add frontend/backend/api decorators to autoload path #44

Merged
merged 1 commit into from
Apr 24, 2020

Conversation

kennyadsl
Copy link
Member

This is required in order to have the decorators correctly unloaded when we reload the application (calling reload! in a Rails console or on each request in development).

@kennyadsl kennyadsl requested a review from aldesantis April 22, 2020 16:02
This is required in order to have the decorators correctly
unloaded when we reload the application (calling reload! in
console or on each request in development).
@kennyadsl kennyadsl force-pushed the fix-require-decorator branch from e6974db to 04ecca6 Compare April 22, 2020 16:11
@kennyadsl kennyadsl changed the title Add subcomponent decorators to autoload path Add frontend/backend/api decorators to autoload path Apr 22, 2020
Copy link
Contributor

@DanielePalombo DanielePalombo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@AlessioRocco AlessioRocco left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Copy link
Member

@spaghetticode spaghetticode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kennyadsl thanks!

@aldesantis aldesantis merged commit df6b9ac into solidusio:master Apr 24, 2020
@aldesantis aldesantis deleted the fix-require-decorator branch April 24, 2020 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants