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

[WIP](Depends on #13060) Dto collection save inventory specs for skeletal refresh #13049

Conversation

Ladas
Copy link
Contributor

@Ladas Ladas commented Dec 8, 2016

Dto collection save inventory specs for skeletal refresh

@Ladas
Copy link
Contributor Author

Ladas commented Dec 8, 2016

@miq-bot assign @agrare

@Ladas Ladas force-pushed the dto_collection_save_inventory_specs_for_skeletal_refresh branch from 2122041 to a8322d6 Compare December 8, 2016 12:30
Ladas added 3 commits December 8, 2016 15:39
Introduce only_update for a DtoCollection, in the sace where
we want to skip all deletes and creates and update only existing
records.
Use create_allowed? and update_allowed? when saving Dtos
Refactor comparing method of collections, so we can easily use
it when debugging
@Ladas Ladas force-pushed the dto_collection_save_inventory_specs_for_skeletal_refresh branch from a8322d6 to 4af7048 Compare December 8, 2016 14:39
@Ladas Ladas changed the title [WIP](Depends on #13018) Dto collection save inventory specs for skeletal refresh Dto collection save inventory specs for skeletal refresh Dec 8, 2016
Ladas added 3 commits December 8, 2016 16:25
A foreign key can be a part of index of the DtoCollection
Enhance Keypair spec data for a skeletal refresh, so we can
separate relations and data
Spec for Dto saving with skeletal strategy
@Ladas Ladas force-pushed the dto_collection_save_inventory_specs_for_skeletal_refresh branch from 15f559b to bf4614d Compare December 8, 2016 15:25
@Ladas Ladas changed the title Dto collection save inventory specs for skeletal refresh [WIP](Depends on #13060) Dto collection save inventory specs for skeletal refresh Dec 8, 2016
@chessbyte chessbyte added the wip label Dec 8, 2016
@miq-bot
Copy link
Member

miq-bot commented Dec 9, 2016

Checked commits Ladas/manageiq@808cbaa~...bf4614d with ruby 2.2.5, rubocop 0.37.2, and haml-lint 0.16.1
2 files checked, 12 offenses detected

spec/models/manager_refresh/save_inventory/graph_of_dto_collections_skeletal_refresh_spec.rb

@miq-bot
Copy link
Member

miq-bot commented Jan 19, 2017

This pull request is not mergeable. Please rebase and repush.

@agrare
Copy link
Member

agrare commented Sep 27, 2017

Closing since this has been in conflict for 9 months :)

@agrare agrare closed this Sep 27, 2017
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.

4 participants