Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Skróty (ma_wpnable, me_wpnable, ra_wpnable) zastosowałem jedynie ze względu na:
ogólnie nie jestem w pełni zadowolony z zaproponowanego przez siebie rozwiązania ze względu na robienie dodatkowych klas (podtypy samurajów i husarzy), ale chociaż poćwiczyłem polimorfizm. Zrobiłbym to raczej jako cecha obiektu (i tak też kombinowałem między commitami).
Jest problem z seedowaniem i serializacją broni - podejrzewam, że najpierw muszę zapanować nad czasem tworzenia broni i łączeniem jej z postaciami. Drugi krok to poprawa serializacji, prawdopodobnie osobno dla każdego podtypu broni. Błędy, w zależności od podejścia:
ActiveRecord::RecordInvalid: Validation failed: Weaponable must exist
ActiveModel::MissingAttributeError: can't write unknown attribute 'me_wpnable_id'
ActiveModel::UnknownAttributeError: unknown attribute 'me_wpnable_id' for MeleeWeapon.
Reszta powinna działać.
Sentry.io działa.