Skip to content

Conversation

@annavik
Copy link
Member

@annavik annavik commented Apr 1, 2025

See #789 for background.

I tried to include project_id as a param, but it's not enough. We also need some backend fix here! Can you help @mohamedelabbas1996?

@netlify
Copy link

netlify bot commented Apr 1, 2025

Deploy Preview for antenna-preview canceled.

Name Link
🔨 Latest commit 0dc70fd
🔍 Latest deploy log https://app.netlify.com/sites/antenna-preview/deploys/67f6f72ee6ebd80008bfd513

@annavik annavik marked this pull request as ready for review April 8, 2025 07:30
Copy link
Member Author

@annavik annavik left a comment

Choose a reason for hiding this comment

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

Thanks a lot @mohamedelabbas1996 for the fix and also for adding tests! I did manual testing from UI for roles superadmin, basic member, identifier and project manager. Works well! 👌

Can't approve the PR since I opened it, but I approve! ✅

@mihow mihow merged commit 0a608a3 into main Apr 9, 2025
6 checks passed
@mihow mihow deleted the fix/upload-captures branch April 9, 2025 22:44
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.

4 participants