Skip to content

Support generating views in rails engine#357

Open
sullerandras wants to merge 2 commits into
scenic-views:mainfrom
sullerandras:support-rails-engine
Open

Support generating views in rails engine#357
sullerandras wants to merge 2 commits into
scenic-views:mainfrom
sullerandras:support-rails-engine

Conversation

@sullerandras
Copy link
Copy Markdown

I ran into the issue that was described in #147. This PR seems to solve the issue.

Possible related issues:

I got uninitialized constant error:
```
/Users/andras/projects/scenic/lib/generators/scenic/view/view_generator.rb:99:in `previous_definition': uninitialized constant Scenic::Definition (NameError)
```

The requires fixed the issue for me.

def views_directory_path
@views_directory_path ||= Rails.root.join("db", "views")
@views_directory_path ||= (Rails.root || Pathname.new(Dir.pwd)).join("db", "views")
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Metrics/LineLength: Line is too long. [91/80]

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.

1 participant