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

[Global Styles] Implements Global Styles API Call #1940

Merged
merged 12 commits into from
Jun 29, 2021
Merged

Conversation

antonis
Copy link

@antonis antonis commented Apr 2, 2021

Fixes: wordpress-mobile/gutenberg-mobile#3163

gutenberg PR: WordPress/gutenberg#30544
gutenberg-mobile PR: wordpress-mobile/gutenberg-mobile#3459
WordPress-Android PR: wordpress-mobile/WordPress-Android#14597

Description

This PR adds support for the new Global styles API endpoint.

To Test

Use the app from the Android PR, turn on the GlobalStyleSupportFeatureConfig (Me > App Settings > Test feature configuration) and follow the test instructions on:

Optionally:

@antonis antonis self-assigned this Apr 2, 2021
@antonis antonis force-pushed the add/3163-GSS branch 3 times, most recently from 4a0fdab to bdfe7cf Compare May 4, 2021 17:22
@antonis antonis changed the title [DO NOT MERGE] Implements Global Styles API Call Implements Global Styles API Call May 4, 2021
@antonis antonis changed the title Implements Global Styles API Call [Global Styles] Implements Global Styles API Call May 5, 2021
@antonis antonis force-pushed the add/3163-GSS branch 2 times, most recently from 003819a to f3cb095 Compare May 26, 2021 13:16
@antonis antonis requested review from chipsnyder and geriux May 27, 2021 12:11
@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Jun 17, 2021

You can trigger optional UI/connected tests for these changes by visiting CircleCI here.

Antonis Lilis added 3 commits June 23, 2021 08:51
# Conflicts:
#	fluxc/src/main/java/org/wordpress/android/fluxc/persistence/WellSqlConfig.kt
@antonis antonis marked this pull request as ready for review June 28, 2021 12:48
Copy link
Contributor

@chipsnyder chipsnyder left a comment

Choose a reason for hiding this comment

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

Changes look good Nice work @antonis 👍

I only had time to give it a smoke test but I'll do a more thorough test before we fully turn the feature on.

@antonis antonis merged commit 2d9132d into develop Jun 29, 2021
@antonis antonis deleted the add/3163-GSS branch June 29, 2021 05:36
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.

2 participants