{"payload":{"feedbackUrl":"https://github.com/orgs/community/discussions/53140","repo":{"id":673478484,"defaultBranch":"master","name":"appwrite","ownerLogin":"safwanyp","currentUserCanPush":false,"isFork":true,"isEmpty":false,"createdAt":"2023-08-01T18:03:54.000Z","ownerAvatar":"https://avatars.githubusercontent.com/u/19154530?v=4","public":true,"private":false,"isOrgOwned":false},"refInfo":{"name":"","listCacheKey":"v0:1690916289.0","currentOid":""},"activityList":{"items":[{"before":"06570a081a3832b94ee6e5307002d55738be079c","after":"a7094310d7e7b6299976c7e2d51bdd825b1b570d","ref":"refs/heads/master","pushedAt":"2023-08-07T07:35:01.000Z","pushType":"push","commitsCount":5,"pusher":{"login":"safwanyp","name":"Safwan Parkar","path":"/safwanyp","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/19154530?s=80&v=4"},"commit":{"message":"Merge pull request #5550 from Suven-p/feat-5542-validate-x-appwrite-id-header-for-CreateFile\n\nValidate value of x-appwrite-id header","shortMessageHtmlLink":"Merge pull request appwrite#5550 from Suven-p/feat-5542-validate-x-ap…"}},{"before":"fcd4a69ad1b3db67c6d89f02c5f230c2038a4b35","after":"06570a081a3832b94ee6e5307002d55738be079c","ref":"refs/heads/master","pushedAt":"2023-08-05T06:48:30.000Z","pushType":"push","commitsCount":13,"pusher":{"login":"safwanyp","name":"Safwan Parkar","path":"/safwanyp","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/19154530?s=80&v=4"},"commit":{"message":"Merge pull request #5928 from safwanyp/fix-5517-fix-stale-team-memberships-on-user\n\nFixed bug where memberships remained after a team is deleted","shortMessageHtmlLink":"Merge pull request appwrite#5928 from safwanyp/fix-5517-fix-stale-tea…"}},{"before":"0295c6ec1b2919385e516d4b86441fee5ff8e9df","after":"4a9af13f167e938da417bf2a1eedf6cb39215e99","ref":"refs/heads/fix-5517-fix-stale-team-memberships-on-user","pushedAt":"2023-08-04T19:41:33.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"safwanyp","name":"Safwan Parkar","path":"/safwanyp","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/19154530?s=80&v=4"},"commit":{"message":"run formatter and linter\n\nRun the composer format and lint commands, which I forgot to run before.","shortMessageHtmlLink":"run formatter and linter"}},{"before":"451f4bee19d31a1408049885a2f8fe6d3fc3faa5","after":"0295c6ec1b2919385e516d4b86441fee5ff8e9df","ref":"refs/heads/fix-5517-fix-stale-team-memberships-on-user","pushedAt":"2023-08-04T19:17:45.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"safwanyp","name":"Safwan Parkar","path":"/safwanyp","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/19154530?s=80&v=4"},"commit":{"message":"improve test by removing user creation loop\n\nThe CREATE TEAM MEMBERSHIP endpoint requires the email of the user to be added to the team. If the user does not exist in the project, a new user is created with the specified email and added to the team.\n\nThe first version of the test creates 5 users, and then adds them to the newly created team. This process is more streamlined now, by using the CREATE TEAM MEMBERSHIPS behaviour to create a user on the go and create a membership for them immediately after.\n\nI also change the way I add user IDs to the array, by using the shorthand notation instead of the `array_push` function.","shortMessageHtmlLink":"improve test by removing user creation loop"}},{"before":"06e1191063435bb184a356aa571827301554f623","after":"451f4bee19d31a1408049885a2f8fe6d3fc3faa5","ref":"refs/heads/fix-5517-fix-stale-team-memberships-on-user","pushedAt":"2023-08-04T11:40:46.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"safwanyp","name":"Safwan Parkar","path":"/safwanyp","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/19154530?s=80&v=4"},"commit":{"message":"Fixed an incorrect test for team deletion\n\nThis commit contains changes in 3 places.\n\n- First I changed the placement of an informative comment in the DELETE TEAM controller, and moved it outside of the loop. I did this when Steven pointed out that the behaviour I describe in the comment is for the whole loop.\n\n- The second change is the removal of my first test. I was facing quite a few issues with creating users in the test, and ended up using the CREATE TEAM MEMBERSHIP to perform 2 actions at once -> create a new user if one doesn't exist with the provided email, and create a membership for the user. Before this approach, I had quite a bit of code that didn't work, and it seems like I removed some things that weren't supposed to be removed, and didn't change variable names where necessary. Anyway, I figured that the problem has something to do with the user being created on the client side, so I moved the test to the server side.\n\n- The new test I implemented does the same thing as my previous failed test, but in more detailed and distinct steps. The test first creates 5 new users inside of a loop, and pushes each new user's ID to an array called 'new_users' if the response is as expected. Then a new team is created. The next step is to create memberships for all 5 users. If all these steps pass, the new team that was just created, is deleted, and we check to make sure the new users have 0 team memberships each.\n\nFormatter and linter showed no errors. Tests were successful on localhost.","shortMessageHtmlLink":"Fixed an incorrect test for team deletion"}},{"before":"d142620c5e9ec7f3125e6d39f0c4c7d302bfc62a","after":"06e1191063435bb184a356aa571827301554f623","ref":"refs/heads/fix-5517-fix-stale-team-memberships-on-user","pushedAt":"2023-08-03T20:34:04.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"safwanyp","name":"Safwan Parkar","path":"/safwanyp","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/19154530?s=80&v=4"},"commit":{"message":"added test","shortMessageHtmlLink":"added test"}},{"before":"a14d89eb3cc3cc7fb28d154682b33565f990d0b1","after":"d142620c5e9ec7f3125e6d39f0c4c7d302bfc62a","ref":"refs/heads/fix-5517-fix-stale-team-memberships-on-user","pushedAt":"2023-08-03T18:08:30.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"safwanyp","name":"Safwan Parkar","path":"/safwanyp","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/19154530?s=80&v=4"},"commit":{"message":"invalidate cached document of user\n- cache caused stale data in memberships","shortMessageHtmlLink":"invalidate cached document of user"}},{"before":"2bc2061f091eb4e3ba358292057dac1db02c4273","after":"a14d89eb3cc3cc7fb28d154682b33565f990d0b1","ref":"refs/heads/fix-5517-fix-stale-team-memberships-on-user","pushedAt":"2023-08-03T17:39:22.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"safwanyp","name":"Safwan Parkar","path":"/safwanyp","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/19154530?s=80&v=4"},"commit":{"message":"remove error-causing condition","shortMessageHtmlLink":"remove error-causing condition"}},{"before":"c5233d9ece9f9c0a82c6121a367d3f63ce032dd2","after":"2bc2061f091eb4e3ba358292057dac1db02c4273","ref":"refs/heads/fix-5517-fix-stale-team-memberships-on-user","pushedAt":"2023-08-02T14:44:59.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"safwanyp","name":"Safwan Parkar","path":"/safwanyp","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/19154530?s=80&v=4"},"commit":{"message":"fix showing of stale team memberships","shortMessageHtmlLink":"fix showing of stale team memberships"}},{"before":"a45c62ab24c8a9e415a46290e968a70a3741bc12","after":"c5233d9ece9f9c0a82c6121a367d3f63ce032dd2","ref":"refs/heads/fix-5517-fix-stale-team-memberships-on-user","pushedAt":"2023-08-02T08:18:26.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"safwanyp","name":"Safwan Parkar","path":"/safwanyp","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/19154530?s=80&v=4"},"commit":{"message":"removed unnecessary code","shortMessageHtmlLink":"removed unnecessary code"}},{"before":"0fe59d457b08fbbf588a128e8a410e618d48308e","after":"a45c62ab24c8a9e415a46290e968a70a3741bc12","ref":"refs/heads/fix-5517-fix-stale-team-memberships-on-user","pushedAt":"2023-08-01T19:24:49.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"safwanyp","name":"Safwan Parkar","path":"/safwanyp","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/19154530?s=80&v=4"},"commit":{"message":"run composer scripts","shortMessageHtmlLink":"run composer scripts"}},{"before":null,"after":"0fe59d457b08fbbf588a128e8a410e618d48308e","ref":"refs/heads/fix-5517-fix-stale-team-memberships-on-user","pushedAt":"2023-08-01T18:58:09.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"safwanyp","name":"Safwan Parkar","path":"/safwanyp","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/19154530?s=80&v=4"},"commit":{"message":"fixed stale team memberships on user","shortMessageHtmlLink":"fixed stale team memberships on user"}}],"hasNextPage":false,"hasPreviousPage":false,"activityType":"all","actor":null,"timePeriod":"all","sort":"DESC","perPage":30,"cursor":"djE6ks8AAAADZT9FlAA","startCursor":null,"endCursor":null}},"title":"Activity · safwanyp/appwrite"}