Add RichText field type

[littleforest]

This adds a RichText field type, completing the work started in #1660.

[littleforest]

Will look into the failing tests, some of them are not failing for me locally :(

[pablobm]

I had a look. It appears to be order-dependent.

Specifically, when I run en example from spec/generators/routes_generator_spec.rb first, followed by spec/administrate/views/fields/rich_text/_form_spec.rb, the latter fails. Looking into it a bit more, I have narrowed it down to the reset_routes call in the after hook, and in turn to the Rails.application.routes.clear! within it:

administrate/spec/support/generator_spec_helpers.rb

Lines 38 to 41 in 852b62c

	def reset_routes
	Rails.application.routes.clear!
	load "spec/example_app/config/routes.rb"
	end

So it looks that, whatever happens there, it deletes the rails_direct_upload_url helper, making the example fail.

Thoughts?

[santosh-1987]

Are we planning to release this anytime soon - Need this badly for actiontext

[nickcharlton]

@littleforest, are you able to pick up from @seanpdoyle's comments here?

(If not, I can do so!)

[littleforest]

@nickcharlton I will see if I can find time to work on this second week of January. Might go faster if I can find time to pair with you or @seanpdoyle .

[nickcharlton]

@littleforest, sorry, only getting back to this one now. How did you get on? Feel free to drop some time on my calendar if you've like to pair on this.

[seanpdoyle]

@littleforest the asset changes merged as part of #2397 have introduced merge conflicts.

If you're interested, I'm willing to give resolving those conflicts a try and force-pushing to this branch.

If not, and you'd rather handle that yourself, that's great too!

[littleforest]

I'm willing to give resolving those conflicts a try and force-pushing to this branch

That would be amazing! Thank you.

[littleforest]

@seanpdoyle @nickcharlton @pablobm this is ready for re-review.

[nickcharlton]

Looks good!

Just one thought on added dependencies, but we should be able to merge this shortly.

[nickcharlton]

I've just rebased, and I'll let the tests run before merging this in.

For those following along, it'll go out with our next v1.0 beta release, which I hope to do soon, but I'm behind things at the moment so it might be a while.

[nickcharlton]

Bundle audit is failing because of a security vulnerability in actiontext that we've not merged yet.

[nickcharlton]

And merged. Thanks @sedubois for the original work, and @littleforest for consistently pushing this through!

[kaka-ruto]

it'll go out with our next v1.0 beta release

Hi @nickcharlton . Thanks for the work in this gem. Is the above still the case? As I don't see anywhere in the v1.0 releasementioning the addition of the RichText field type

[nickcharlton]

It's waiting on a second beta release, where it'll be included then. I'm going to try and do this today, so keep an eye out on the releases.