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

feat: set MEILISEARCH_ENABLED in MFE_CONFIG [FC-0040] #1

Merged
merged 1 commit into from
Apr 1, 2024
Merged

feat: set MEILISEARCH_ENABLED in MFE_CONFIG [FC-0040] #1

merged 1 commit into from
Apr 1, 2024

Conversation

rpenido
Copy link
Member

@rpenido rpenido commented Mar 29, 2024

Description

This PR passes the MEILISEARCH_ENABLED settings to the MFE_CONFIG variable to be used in the MFEs.

More Information

Part of:


Internal ref: FAL-3693

@rpenido rpenido marked this pull request as ready for review March 29, 2024 13:43
@rpenido rpenido changed the title feat: set MEILISEARCH_ENABLED in MFE_CONFIG feat: set MEILISEARCH_ENABLED in MFE_CONFIG [FC-0040] Mar 29, 2024
Copy link

@yusuf-musleh yusuf-musleh left a comment

Choose a reason for hiding this comment

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

👍 It's working well @rpenido !

  • I tested this: Tested as part of related PR
  • I read through the code
  • I checked for accessibility issues
  • Includes documentation

@bradenmacdonald bradenmacdonald merged commit dfa7ac2 into open-craft:main Apr 1, 2024
@bradenmacdonald
Copy link
Member

@rpenido If you can push your changes to the open-craft fork, it's much easier to test than on your own fork.

@rpenido
Copy link
Member Author

rpenido commented Apr 1, 2024

@rpenido If you can push your changes to the open-craft fork, it's much easier to test than on your own fork.

@bradenmacdonald
Sorry! I forgot to mention it!
I don't have permission to create a branch at the open-craft fork.

@rpenido rpenido deleted the rpenido/fal-3693-pass-meilisearch-enabled-env-to-mfe-config branch April 1, 2024 17:20
@bradenmacdonald
Copy link
Member

@rpenido Sorry about that. Fixed :)

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