Skip to content

Conversation

hb0730
Copy link
Contributor

@hb0730 hb0730 commented Sep 9, 2025

fix #3701

@binarywang binarywang requested a review from Copilot September 14, 2025 09:06
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for Redis Template configuration in the multi-miniapp Spring Boot starter, addressing issue #3701. It enables users to configure mini-program services using Spring's StringRedisTemplate instead of being limited to Jedis or Redisson.

  • Adds a new configuration class WxMaInRedisTemplateConfiguration that implements Redis Template-based storage
  • Integrates the new configuration into the auto-configuration system

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
WxMaInRedisTemplateConfiguration.java New configuration class implementing Redis Template storage for multi-miniapp setup
WxMaMultiServiceConfiguration.java Updated import list to include the new Redis Template configuration

@Configuration
@ConditionalOnProperty(
prefix = WxMaMultiProperties.PREFIX + ".config-storage", name = "type", havingValue = "redis_template",
matchIfMissing = true
Copy link
Preview

Copilot AI Sep 14, 2025

Choose a reason for hiding this comment

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

The matchIfMissing = true should likely be false for a Redis Template configuration. When no storage type is explicitly configured, it shouldn't default to Redis Template which requires StringRedisTemplate bean to be available.

Suggested change
matchIfMissing = true
matchIfMissing = false

Copilot uses AI. Check for mistakes.

@hb0730 hb0730 requested a review from Copilot September 14, 2025 09:24
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

@binarywang binarywang merged commit c8d84da into binarywang:develop Sep 15, 2025
@binarywang binarywang added this to the 4.7.8 milestone Sep 16, 2025
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.

多小程序好像并未适配spring redis_template
2 participants