Skip to content

Release - 2.1.0#82

Merged
tarecord merged 6 commits into
mainfrom
release/2.1.0
Aug 22, 2023
Merged

Release - 2.1.0#82
tarecord merged 6 commits into
mainfrom
release/2.1.0

Conversation

@tarecord
Copy link
Copy Markdown
Member

No description provided.

@tarecord tarecord self-assigned this Aug 21, 2023
@tarecord tarecord requested a review from ChrisMKindred August 22, 2023 16:13
Copy link
Copy Markdown
Contributor

@ChrisMKindred ChrisMKindred left a comment

Choose a reason for hiding this comment

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

Looks good, I have one quick comment, but feel free to merge and move that to a new backlogged item if it needs to change.

define( 'WP_CONTENT_URL', sprintf( '%s/wp-content', WP_HOME ) );
define( 'WP_CONTENT_DIR', __DIR__ . '/wp-content' );
define( 'TELEMETRY_SERVER_URL', app_env( 'TELEMETRY_SERVER_URL', 'https://telemetry-dev.stellarwp.com/api/v1' ) );
define( 'TELEMETRY_SERVER_URL', app_env( 'TELEMETRY_SERVER_URL', 'https://telemetry-server.lndo.site/api/v1' ) );
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should we update this naming to match the conversation here?

@tarecord tarecord merged commit 0678280 into main Aug 22, 2023
@tarecord tarecord deleted the release/2.1.0 branch August 22, 2023 17:03
@tarecord tarecord restored the release/2.1.0 branch August 24, 2023 14:04
@tarecord tarecord deleted the release/2.1.0 branch August 24, 2023 14:05
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.

3 participants