Skip to content
This repository was archived by the owner on Jan 26, 2024. It is now read-only.

Helped Migrate to Appwrite Version 0.15.^ and Appwrite SDK 6.0.0#2

Merged
brandonroberts merged 9 commits intoappwrite:mainfrom
kekavc24:migrate
Jul 27, 2022
Merged

Helped Migrate to Appwrite Version 0.15.^ and Appwrite SDK 6.0.0#2
brandonroberts merged 9 commits intoappwrite:mainfrom
kekavc24:migrate

Conversation

@kekavc24
Copy link
Copy Markdown
Contributor

Helped :

  • Migrate the repo to use Appwrite version 0.15.^ and Appwrite SDK 6.0.0
  • Fixed some but not all linter issues. I left the various unused variables. I assumed you had planned to use them in future.

@stnguyen90
Copy link
Copy Markdown

Thanks so much for your contribution! 🙏 Let me look into who can review this so we can get it merged in!

Copy link
Copy Markdown

@brandonroberts brandonroberts left a comment

Choose a reason for hiding this comment

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

Overall LGTM, just needs the bucketId to be updated. Tested and working locally

Comment thread lib/providers/watchlist.dart Outdated

class WatchListProvider extends ChangeNotifier {
final String _collectionId = "watchlists";
static const String _bucketId = "Your bucket ID";
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The _bucketId is "default"

Comment thread lib/providers/entry.dart Outdated

Map<String, Uint8List> _imageCache = {};
static const String _collectionId = "movies";
static const String _bucketId = "Your bucket ID";
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Should be "default"

@kekavc24 kekavc24 requested a review from brandonroberts July 27, 2022 19:26
Copy link
Copy Markdown

@brandonroberts brandonroberts left a comment

Choose a reason for hiding this comment

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

LGTM

@brandonroberts brandonroberts merged commit 601eaae into appwrite:main Jul 27, 2022
@brandonroberts
Copy link
Copy Markdown

Thanks @kekavc24! Nice work!

@kekavc24
Copy link
Copy Markdown
Contributor Author

No problem 💯 Thought I'd help since maybe you guys are busy making Appwrite Awesome 😜😎

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