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

Update storage and avatars services to include methods to get URLs #74

Closed
wants to merge 2 commits into from

Conversation

stnguyen90
Copy link
Contributor

@stnguyen90 stnguyen90 commented Jul 22, 2022

Issue: #71

SDK Generator PR: appwrite/sdk-generator#495

@stnguyen90 stnguyen90 self-assigned this Jul 22, 2022
@stnguyen90
Copy link
Contributor Author

stnguyen90 commented Jul 22, 2022

@lohanidamodar, you can focus on 9a770bc to see the changes to the storage and avatars services. Hopefully, this gives you an idea of what I was thinking we would provide. We can finalize the name of the methods and docs before merging.

@stnguyen90 stnguyen90 marked this pull request as draft July 22, 2022 22:12
lib/services/storage.dart Outdated Show resolved Hide resolved
@stnguyen90 stnguyen90 force-pushed the feat-71-method-generate-file-url branch from 6a75c0c to 9a770bc Compare July 22, 2022 22:42
@Shiba-Kar
Copy link

Done This

appwrite:
    git:
      url: https://github.com/appwrite/sdk-for-flutter.git
      ref: feat-71-method-generate-file-url

getting this error

('../../../../../AppData/Local/Pub/Cache/git/sdk-for-flutter-9a770bc75904727660acc7b58a1a9344e8ae68dc/lib/src/client.dart').
[        ] Try correcting the name to the name of an existing getter, or defining a getter or field named 'endpoint'.
[        ]         final uri = Uri.parse(client.endpoint + path).replace(queryParameters: params);

We want to get the image Url its a must have feature.

@stnguyen90
Copy link
Contributor Author

We want to get the image Url its a must have feature.

@Shiba-Kar, this is a pretty stale branch and only a POC. I would suggest building the URL manually instead of trying to use this branch.

@stnguyen90 stnguyen90 closed this Mar 20, 2023
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