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

Split existing configuration guide #2488

Merged
merged 10 commits into from
Mar 10, 2018

Conversation

benjaminwil
Copy link
Contributor

I've cleaned up and rewritten some of the existing configuration guide. More noticeably, I've split the existing guide up into a number of new articles that now reside in different subdirectories.

  • The getting-started directory now has a stub about why you might want to create a private Solidus fork.
  • A new extensions directory houses an article about installing extensions and (for now) an article about decorators (though I think this needs another home).
  • A new views directory has an article about overriding views.
  • A new preferences directory has articles about site-wide preferences configuration, class extension points, and creating model-specific preferences.

None of this is entirely new content, but used to all live together in one article. Splitting things up into more focused directories will hopefully help developers find the information they need faster as the Solidus documentation continues to grow.

This is part a larger project to improve Solidus's documentation. See this gist with the high-level table of contents. Where and how this documentation will exist is still up for discussion.

Solidus extensions can be installed in a few simple steps:

1. Add the extension gem to your project's `Gemfile`.
2. Run `bundle install to install the gem and its dependencies.
Copy link
Contributor

@swcraig swcraig Jan 5, 2018

Choose a reason for hiding this comment

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

I think you are missing an ending ` for bundle install.

@benjaminwil benjaminwil force-pushed the split_configuration_guide branch 2 times, most recently from a4f1405 to 79fdb90 Compare January 5, 2018 22:26
@benjaminwil benjaminwil force-pushed the split_configuration_guide branch from 3fb863e to c8acd33 Compare January 30, 2018 11:53
@benjaminwil benjaminwil force-pushed the split_configuration_guide branch from c8acd33 to b153cef Compare February 8, 2018 19:11
Copy link
Member

@adammathys adammathys left a comment

Choose a reason for hiding this comment

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

Just a couple of minor things.

gem 'solidus', git: 'https://github.com/my_account/solidus.git', branch: "my-new-feature"
```

If you think your feature (or fix) if of interest to the wider Solidus
Copy link
Member

Choose a reason for hiding this comment

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

(or fix) if of interest

@@ -0,0 +1,23 @@
# Private Solidus fork
Copy link
Member

Choose a reason for hiding this comment

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

It might be a little bit misleading to use the word "private" here. It has a specific connotation when associated to a GitHub fork. (Technically, there's considerably more work involved in making a private fork of Solidus, since making a private fork of a public repo isn't functionality GitHub provides.)


## Define new preferences

You can define preferences for a model within the model itself:
Copy link
Member

Choose a reason for hiding this comment

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

We should probably mention that a column needs to be added to the model to persist the preferences.

class MyMigration < ActiveRecord::Migration[5.0]
  def change
    add_column :my_table, :preferences, :text
  end
end

# App configuration

Solidus includes many preferences with default settings that are appropriate for
typical stores. For a list of Solidus's preferences their default values, see
Copy link
Member

Choose a reason for hiding this comment

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

Solidus's preferences and their default values . . .

@benjaminwil
Copy link
Contributor Author

Thanks for the review @adammathys. I've committed changes for all your comments.

If you could approve that would be great. 👍

benjaminwil and others added 10 commits February 27, 2018 14:18
After speaking with jhawthorne about it, it seemed right to mention
that we do not support extending the `Spree::AppConfiguration` object,
and encourage developers to try using new model preferences to get
the functionality they desire. (Although that may not always help
them.)
Thanks to adammathys for his review of this content.
@benjaminwil benjaminwil force-pushed the split_configuration_guide branch from f95a6b6 to 5d4cdc2 Compare February 27, 2018 22:19
@kennyadsl kennyadsl merged commit b0bb197 into solidusio:master Mar 10, 2018
@benjaminwil benjaminwil deleted the split_configuration_guide branch April 30, 2018 18:29
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