Skip to content
This repository was archived by the owner on Sep 29, 2023. It is now read-only.

Conversation

Meldiron
Copy link
Contributor

@Meldiron Meldiron commented Apr 13, 2022

PR appwrite/appwrite#3074 introduces new certificate features that aren't documented. This PR makes sure all logic done in Appwrite is properly documented, and people are aware of all options and processes.

CleanShot 2022-04-13 at 15 39 49

Copy link
Contributor

@gewenyu99 gewenyu99 left a comment

Choose a reason for hiding this comment

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

👋 Hi Matej, minor English suggestions.

Otherwise very clear and well written <3333

Copy link
Contributor

@gewenyu99 gewenyu99 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@christyjacob4 christyjacob4 left a comment

Choose a reason for hiding this comment

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

Please address the comments

Meldiron and others added 2 commits April 22, 2022 14:03
Co-authored-by: Christy Jacob <christyjacob4@gmail.com>
@christyjacob4 christyjacob4 merged commit 44cfde7 into main May 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants