Skip to content
This repository has been archived by the owner on Jul 16, 2022. It is now read-only.

Updated progress for function-demos #127

Closed
wants to merge 4 commits into from
Closed

Updated progress for function-demos #127

wants to merge 4 commits into from

Conversation

noviicee
Copy link

@noviicee noviicee commented Oct 5, 2021

refer #126
@Meldiron
Copy link
Contributor

Meldiron commented Oct 6, 2021

@noviicee Please add screenshots of successful execution and execution logs from the Appwrite console. For example, look at this PR.
@noviicee
Copy link
Author

noviicee commented Oct 6, 2021

@Meldiron this was the PR I made earlier where I just updated the Readme. But as you said that having a workflow for updating the progress of the demo functions would be better, so now I am working on implementing that.
@Meldiron
Copy link
Contributor

I see.. Hmm, does this actually work? 🤔 Is there any way to see this in action before merging? I cannot see where in the action we provide the structure of the table we want it to create.
Meldiron
Meldiron suggested changes Oct 27, 2021
View reviewed changes
Copy link
Contributor

@Meldiron Meldiron left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Please include test results in any form... Screenshot / video / rest repository
@christyjacob4
Copy link
Member

@noviicee Closing this since there has been no activity on this in a while. Feel free to reopen it when you are ready to start working on it again 😄
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
3 participants
@noviicee @Meldiron @christyjacob4