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

Endpoint for server info #835

Merged
merged 2 commits into from
Sep 29, 2024
Merged

Endpoint for server info #835

merged 2 commits into from
Sep 29, 2024

Conversation

khorshuheng
Copy link
Collaborator

There are a few scenarios which it will be helpful for a client to be aware of the functionality currently supported by AppFlowy Cloud:

  1. A user is using self hosting an older version of AppFlowy Cloud, which does not support some of the newer functionality introduced by the newer client. Having a server info endpoint will allow the client to either provide a fallback, or provide appropriate error message to the user.

  2. A user is using an older version of AppFlowy client, which relies on functionality that is already deprecated by AppFlowy Cloud. By checking the features supported by the server, the client can prompt the user to upgrade the client version.

@khorshuheng
Copy link
Collaborator Author

The other alternative to returning a supported feature list, would be to return the server version, and the minimum supported client version. It's not possible to do this at the moment, however, because our server package version is always 0.1.0.

@appflowy appflowy merged commit 47a1aae into main Sep 29, 2024
9 of 10 checks passed
@appflowy appflowy deleted the endpoint-for-server-info branch September 29, 2024 03:01
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.

2 participants