Skip to content

(#42) migrate acts-as-taggable-on to the demo_plugin#45

Merged
paranoicsan merged 2 commits intomasterfrom
42-refactor-tag-model
Apr 10, 2026
Merged

(#42) migrate acts-as-taggable-on to the demo_plugin#45
paranoicsan merged 2 commits intomasterfrom
42-refactor-tag-model

Conversation

@alexsdev91
Copy link
Copy Markdown
Contributor

#42

@paranoicsan paranoicsan marked this pull request as ready for review April 8, 2026 12:40
Copy link
Copy Markdown
Collaborator

@paranoicsan paranoicsan left a comment

Choose a reason for hiding this comment

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

Good job, thank you. I think we can merge it after some minor updates.

Comment thread db/schema.rb Outdated
Comment thread lib/plugins/plugin_demo/app/models/plugin_demo/tag.rb Outdated
@alexsdev91 alexsdev91 force-pushed the 42-refactor-tag-model branch from 5fc7fa5 to 3c67ce5 Compare April 9, 2026 17:30
@alexsdev91 alexsdev91 force-pushed the 42-refactor-tag-model branch from 3c67ce5 to d36de39 Compare April 9, 2026 17:39
- Add drop_table for old tags/taggings before creating new ones
- Rename migration files and classes to match table names
- Restore acts_as_taggable_on :tags on Resource via plugin setup
- Remove redundant require "acts-as-taggable-on" from models
- Restore where_context scope on PluginDemo::Tag
- Add resource tagging integration spec
- Document shared gem tables and inter-plugin dependencies
- Add ownership comments to models, migrations, and Gemfile

Signed-off-by: Alexander Kuznetsov <paranoic.san@gmail.com>
@paranoicsan paranoicsan merged commit 2948582 into master Apr 10, 2026
3 checks passed
@paranoicsan paranoicsan deleted the 42-refactor-tag-model branch April 10, 2026 08:29
@paranoicsan paranoicsan added dependencies Pull requests that update a dependency file ruby Pull requests that update ruby code labels Apr 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file ruby Pull requests that update ruby code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants