Skip to content

Conversation

@gewenyu99
Copy link

What does this PR do?

The generated model references map['data'], which is not defined because 'data' is not passed in from the API.

Instead, the API passes in user-defined attribute at the same level as system attributes that start with $

See: https://github.com/appwrite/sdk-for-ruby/blob/dd56d3d4a0ff469ab19c7728ac4f508daa7b8d42/lib/appwrite/models/document.rb#L39C1-L40C1

This fixes the issue by dynamically creating this data map from the map directly.

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

(Write your answer here.)

@gewenyu99 gewenyu99 requested a review from abnegate January 18, 2024 21:23
@abnegate abnegate merged commit 0aa6747 into master Jan 19, 2024
@abnegate abnegate deleted the fix-ruby-list-document branch January 19, 2024 01:04
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.

3 participants