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

Remove Flarum synchronization service from the webserver #3165

Merged
merged 9 commits into from
Jan 10, 2025

Conversation

aglinxinyuan
Copy link
Collaborator

@aglinxinyuan aglinxinyuan commented Dec 17, 2024

Remove the redundant Flarum user registration service from the Texera backend, as it is unnecessary. User registration for Flarum can be handled directly by calling the Flarum user registration API from the Texera frontend.

This PR does not affect the functionality or lifecycle of either Texera or Flarum.

@aglinxinyuan
Copy link
Collaborator Author

aglinxinyuan commented Dec 18, 2024

@shengquan-ni Please test this branch in the Kubernetes (K8s) setup and confirm that new users can successfully sync their login state with the forum.

@aglinxinyuan aglinxinyuan merged commit d4de38f into master Jan 10, 2025
8 checks passed
@aglinxinyuan aglinxinyuan deleted the xinyuan-fix-forum branch January 10, 2025 22:55
shengquan-ni pushed a commit that referenced this pull request Jan 13, 2025
Remove the redundant Flarum user registration service from the Texera
backend, as it is unnecessary. User registration for Flarum can be
handled directly by calling the Flarum user registration API from the
Texera frontend.

This PR does not affect the functionality or lifecycle of either Texera
or Flarum.
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