Skip to content
This repository was archived by the owner on Mar 11, 2021. It is now read-only.

[WIP] Handle expanded character range slugs better #426

Draft
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

reefdog
Copy link
Contributor

@reefdog reefdog commented Nov 11, 2016

Full description coming. Meanwhile see #315.

`URI.parse` chokes on CGI-unescaped unicode chars. `Addressable::URI.parse` doesn't. We're already using Addressable downstream, so just promote it to first-class required gem and use it.
This smells bad and I would like to rethink it.
# Conflicts:
#	Gemfile
#	Gemfile.lock
# Conflicts:
#	app/controllers/api_controller.rb
#	test/controllers/oembed_controller_test.rb
#	test/lib/statistics_test.rb
@reefdog reefdog marked this pull request as draft May 12, 2020 15:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant