Skip to content

Use optional gems in development app#3362

Merged
josepjaume merged 4 commits intomasterfrom
fix/optional_gems_again
May 9, 2018
Merged

Use optional gems in development app#3362
josepjaume merged 4 commits intomasterfrom
fix/optional_gems_again

Conversation

@deivid-rodriguez
Copy link
Copy Markdown
Contributor

🎩 What? Why?

Currently optional gems (consultations & initiatives) are not used when generating a development app. I think we should probably use them in development?

@mrcasals @oriolgual This might be relevant for your demo.

📌 Related Issues

None.

📋 Subtasks

None.

📷 Screenshots (optional)

None.

@ghost ghost assigned deivid-rodriguez May 8, 2018
@ghost ghost added the status: WIP label May 8, 2018
@oriolgual
Copy link
Copy Markdown
Contributor

I think we should enable it, @josepjaume are you OK with this?

Copy link
Copy Markdown
Contributor

@josepjaume josepjaume left a comment

Choose a reason for hiding this comment

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

Fine for me, but would like @oriolgual to confirm it works for him!

gem "decidim-consultations", path: "decidim-consultations"
gem "decidim-initiatives", path: "decidim-initiatives"
gem "decidim-consultations", path: "."
gem "decidim-initiatives", path: "."
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@oriolgual and I had some back-and-forth with this, as it didn't work for him using . as a relative path. Might have to do with the ruby version manager used. @oriolgual can you confirm this works for you?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I should work because we're doing the same for decidim and decidim-dev, no?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I can't actually remember this 😅

But if it fails only at my computer then it's an issue with my setup, not Decidim's

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Well it might indicate a flaw with the approach. But let's do this!

@deivid-rodriguez
Copy link
Copy Markdown
Contributor Author

deivid-rodriguez commented May 9, 2018

By the way, I think this was the way it originally worked (at least I rememeber seeing initiatives and consultations on new apps), but somehow it got reverted (that was probably me, unintentionally 😳)

@josepjaume josepjaume merged commit f8a0afa into master May 9, 2018
@josepjaume josepjaume deleted the fix/optional_gems_again branch May 9, 2018 14:43
@ghost ghost removed the status: WIP label May 9, 2018
deivid-rodriguez added a commit that referenced this pull request May 21, 2018
* Add a rake task to bundle all gemfiles at once

* Normalize optional gems paths

* Make sure migrations pick the right Gemfile

And thus, optional gem migrations are installed and run.

* Use optional gems in "demo apps"
mrcasals pushed a commit that referenced this pull request May 22, 2018
* Fix dev app's gemfile for external plugins (#3313)

* Correct optional gems location when generating app

Otherwise we get an error bundling it in external plugins since there's
no local folder in the final app:

```
The path `/path/to/decidim/development_app/decidim-consultations` does not exist
```

* Read gemspec only once

* Add a change log entry

* Update ruby to version 2.5.1 (#3293)

* Update Gemfile.lock to update erb_lint to v 0.0.24

* fix erb_lint errors

* update to ruby 2.5.1

* Use ruby 2.5.1's docker image

* Update decidim-generators Gemfile.lock

* Update decidim_app-design Gemfile.lock

* Update decidim_app-design Gemfile.lock (2)

* Use optional gems in development app (#3362)

* Add a rake task to bundle all gemfiles at once

* Normalize optional gems paths

* Make sure migrations pick the right Gemfile

And thus, optional gem migrations are installed and run.

* Use optional gems in "demo apps"
isaacmg410 pushed a commit to CodiTramuntana/decidim that referenced this pull request May 25, 2018
* Add a rake task to bundle all gemfiles at once

* Normalize optional gems paths

* Make sure migrations pick the right Gemfile

And thus, optional gem migrations are installed and run.

* Use optional gems in "demo apps"
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.

3 participants