Drop syncConfigToGalaxy

[davelopez]

Do we still need this? smoke testing...

Potentially fixes #16559

How to test the changes?

• This is a refactoring of components with existing test coverage.

License

• I agree to license these and all my past contributions to the core galaxy codebase under the MIT license.

[dannon]

My guess is likely we still need it -- we use Galaxy.config off the global object all over the place still

[davelopez]

I have a bit of trouble trying to understand what this is actually doing, from what I understand, it is subscribing to changes in the config in the Galaxy.config instance and then "updating" the config in the Pinia store. But I thought the configuration is rather a static object, so this is likely not happening, or am I wrong? 🤔

[dannon]

@davelopez Your read of it is correct, and I just went through all occurences I could find and I think we might actually be able to do this now, which is great news. Tests are passing, so let's get it in early in the cycle here and keep our eyes open for any possible weirdness in dark corners of the app :)

I'd like to get rid of all the rxjs over time, for what it's worth.

[github-actions]

This PR was merged without a "kind/" label, please correct.