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

fix: editorInterface methods do not reset custom sidebar anymore #175

Merged
merged 1 commit into from
Mar 27, 2019

Conversation

suevalov
Copy link
Contributor

@suevalov suevalov commented Mar 26, 2019

Summary

Fixes #173

Description

editorInterface payload can have custom sidebar configuration saved. Currently all related methods just override sidebar configuration. This PR fixes the behavior, so if sidebar configuration isn't overwritten if present in the initial payload.

@suevalov suevalov changed the title [WIP ]fix: editorInterface methods do not reset custom sidebar anymore [WIP] fix: editorInterface methods do not reset custom sidebar anymore Mar 26, 2019
@Khaledgarbaya Khaledgarbaya self-assigned this Mar 27, 2019
@suevalov suevalov force-pushed the add-custom-sidebar-support branch 2 times, most recently from 76b0b7d to 264987d Compare March 27, 2019 10:34
@codecov
Copy link

codecov bot commented Mar 27, 2019

Codecov Report

Merging #175 into master will increase coverage by 3.95%.
The diff coverage is 33.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #175      +/-   ##
==========================================
+ Coverage   84.72%   88.68%   +3.95%     
==========================================
  Files          84       84              
  Lines        2330     2244      -86     
  Branches      272      259      -13     
==========================================
+ Hits         1974     1990      +16     
+ Misses        270      201      -69     
+ Partials       86       53      -33
Impacted Files Coverage Δ
src/lib/entities/content-type.ts 93.97% <33.33%> (-2.28%) ⬇️
src/lib/errors/editorinterfaces-fetching.ts 33.33% <0%> (-6.67%) ⬇️
src/lib/errors/space-access.ts 33.33% <0%> (-6.67%) ⬇️
src/lib/migration-steps/dispatch-proxy.ts 83.33% <0%> (-2.39%) ⬇️
src/lib/action/field-create.ts 100% <0%> (ø) ⬆️
src/lib/intent/content-type-create.ts 100% <0%> (ø) ⬆️
src/lib/intent/field-delete.ts 100% <0%> (ø) ⬆️
src/lib/intent/field-move.ts 100% <0%> (ø) ⬆️
src/lib/migration-steps/first-external-caller.js 100% <0%> (ø) ⬆️
...e-api/validator/schema/field-validations-schema.ts 100% <0%> (ø) ⬆️
... and 18 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2728f82...f6976ff. Read the comment docs.

@suevalov suevalov force-pushed the add-custom-sidebar-support branch from 264987d to c63c4dc Compare March 27, 2019 10:40
@suevalov suevalov force-pushed the add-custom-sidebar-support branch from c63c4dc to f6976ff Compare March 27, 2019 10:44
@suevalov suevalov changed the title [WIP] fix: editorInterface methods do not reset custom sidebar anymore fix: editorInterface methods do not reset custom sidebar anymore Mar 27, 2019
@Khaledgarbaya Khaledgarbaya self-requested a review March 27, 2019 12:18
@Khaledgarbaya
Copy link
Contributor

@suevalov can you add tests to the editor interfaces intent at least to make sure that the tool gives an accurate summary about what will happen

here for example

@suevalov
Copy link
Contributor Author

@Khaledgarbaya I think it's quite accurate at the moment. In this PR we're addressing the problem that the tool does what is not intended to happen (rewrites sidebar config).

There will be an another PR which adds possibility to copy and create sidebar configuration.

Copy link
Contributor

@Khaledgarbaya Khaledgarbaya left a comment

Choose a reason for hiding this comment

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

LGTM

@Khaledgarbaya Khaledgarbaya merged commit e2d5bb2 into master Mar 27, 2019
@Khaledgarbaya Khaledgarbaya deleted the add-custom-sidebar-support branch March 27, 2019 13:13
@phoebeschmidt
Copy link
Contributor

🎉 This PR is included in version 0.17.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

running migration resets custom sidebar
3 participants