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

Adding start_at and end_at to :tagged_with #488

Merged
merged 3 commits into from
Jun 17, 2014

Conversation

felipeclopes
Copy link
Contributor

Adding start_at and end_at to the tagged_with method.

This can be used to boost performance on queries where the taggings table is too big, like in the scenario I had to use.

It reduces drastically the amount of rows that the database have to go through, when using a good criteria.

@seuros seuros added the feature label Mar 15, 2014
@seuros
Copy link
Collaborator

seuros commented Mar 19, 2014

Perfect ! Can you rebase ?

@felipeclopes
Copy link
Contributor Author

Done!

@seuros
Copy link
Collaborator

seuros commented Mar 19, 2014

Tests are failing in Postgresql. (Ignore the Sqlite3 error, i'm working on it)

@seuros seuros modified the milestones: 3.1.0, 3.2.0 Mar 28, 2014
@seuros
Copy link
Collaborator

seuros commented Mar 30, 2014

@felipeclopes Can you rebase ? the sqlite bug was fixed. It will be great if you add and example in the Wiki.

@seuros seuros closed this Mar 30, 2014
@seuros seuros reopened this Mar 30, 2014
@seuros
Copy link
Collaborator

seuros commented Apr 18, 2014

bump

@seuros seuros modified the milestones: 3.3.0, 3.2.0 May 1, 2014
@felipeclopes
Copy link
Contributor Author

bump

@seuros
Copy link
Collaborator

seuros commented Jun 16, 2014

Sorry . I will merge tomorrow

seuros added a commit that referenced this pull request Jun 17, 2014
Adding start_at and end_at to :tagged_with
@seuros seuros merged commit d8358e3 into mbleigh:master Jun 17, 2014
tekniklr pushed a commit to tekniklr/acts-as-taggable-on that referenced this pull request Mar 19, 2021
Adding start_at and end_at to :tagged_with
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants