Skip to content

Conversation

ikraamg
Copy link
Collaborator

@ikraamg ikraamg commented Feb 20, 2025

  • Updates the gem to use the latest version of solidus_dev_support
  • Updates gem to support Solidus v4.5.0 and the main branch.
  • Renames the gem to the correct name (SolidusActsAsTenant) instead of "ActAs"
    • (The command to update solidus_dev_support triggered the rename initiative based on the repo name)
  • Use to_prepare when initializing the overrides to support development reloading.
  • Removes Circle CI config in favour of GH actions (introduced by new solidus_dev_support version)

@ikraamg ikraamg changed the title Update-gem-rails8-and-main Update gem to support Rails 8 and work on Solidus main branch Feb 20, 2025
@ikraamg ikraamg force-pushed the update-gem-rails8-and-main branch from cfccb9f to fbd3eae Compare February 20, 2025 15:27
@ikraamg ikraamg changed the title Update gem to support Rails 8 and work on Solidus main branch Update gem to work on Solidus main branch Feb 20, 2025
@ikraamg ikraamg marked this pull request as ready for review February 20, 2025 17:41
@ikraamg ikraamg force-pushed the update-gem-rails8-and-main branch 8 times, most recently from 41677ab to b4e0e0f Compare March 10, 2025 12:29
ikraamg added 7 commits March 10, 2025 12:32
See: solidusio/solidus@00465da
And: solidusio/solidus@470b90a
Acts_as_tenant adds a validation on belongs_to associations that trigger
errors due to the product and payment method models not having a
delegated association associated. This is because Solidus has some
reader methods overriden to return the association object's value when
the current model returns nil for the overriden method.
- This also updated the gem extention name so this commit does renaming as well
It conflicts with devise so it is not a good default without using the Nebulab fork
This fixes a bug when the development env reloader activates and loses
the overrides added by `setup_tenant_aware_models`
@ikraamg ikraamg force-pushed the update-gem-rails8-and-main branch 6 times, most recently from 4cdf1f7 to 706c16c Compare March 10, 2025 13:24
@ikraamg
Copy link
Collaborator Author

ikraamg commented Mar 10, 2025

@DanielePalombo This is ready now. Thank you for the help!

@ikraamg ikraamg force-pushed the update-gem-rails8-and-main branch from 706c16c to d1ba98b Compare March 10, 2025 13:31
@ikraamg ikraamg merged commit bc43a31 into main Mar 10, 2025
46 of 47 checks passed
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.

2 participants