Wp-calypso: Publish package: `lib/simple-payments`

Created on 5 Mar 2019  路  6Comments  路  Source: Automattic/wp-calypso

Packages Simple Payments [Status] Blocked / Hold

All 6 comments

Is Simple payments code in Calypso classic editor going to be needed in Calypso once Gutenlypso is no more? Can we just move these files to the block folder itself:

From: /client/lib/simple-payments
To: client/gutenberg/extensions/simple-payments

...and bring them over to Jetpack and not have them in Calypso codebase?

@Automattic/lannister thoughts?

Used only in tinymce plugin and related data-layer bits:

image

There's still the Calypso Classic editor, so in terms of ordering we'd need to update usages until we complete a full editor rollout.

If this code is only used in the blocks, then we'll just need to make sure Gutenframe is stable, and that Gutenlypso code has been removed. Probably will happen this week.

Sound about right @rodrigoi ?

There is some code that would be good to share so our Simple Payments experience is consistent:

  • Supported currencies
  • The custom post type slug

The de facto way at this time seems to be by publishing a package.

If the classic editor (and associated Simple Payments code) has a limited future in Calypso, it would be completely viable to duplicate the code for now and let it be cleaned up from Calypso.

p1551889885726100-slack-jetpack-gutenberg

@sirreal Code duplication seems reasonable here to me, let's maybe just leave a comment note that they're not connected.

We can let #31199 handle this.

Was this page helpful?
0 / 5 - 0 ratings