SignUp encoding data to JSON

[kwizner]

Parse-SDK-Flutter/packages/dart/lib/src/objects/parse_user.dart

Line 144 in 6e01d28

Future<ParseResponse> signUp() async {

Hi,

I was trying add some data while registering user, but it doesn't work with DateTime object because DateTime does't has toJson method. And we have to send different format to parse than just DateTime in String.

I paste my solution below to save password to unsavedChanges and use existing method toJson from your library.
Instead of current code

      final Map<String, dynamic> bodyData = _getObjectData();
      bodyData[keyVarPassword] = password;
      ...
      final String body = json.encode(bodyData);

I use this

      _unsavedChanges[keyVarPassword] = password;
      ...
      final String body = json.encode(toJson(forApiRQ: true));

With this body we can create parse user with all data.

Let me know if it's clear. If not I will come back with more info.

[fischerscode]

Just to clarify:
You want to create a ParseUser object, then add extra data (esp. a date) and then sign up?
And in order to solve this, you replaced this by this.

This change sounds reasonable, as ParseObject.update() as well as ParseObject.create() are using the toJson method.

[kwizner]

Exactly.
You can create ParseUser object now with some simple data using json encoding from Flutter, but it doesn't with Date!

Don't forget to add password to _unsavedChanges, because you can't use bodyData like before

[fischerscode]

but it doesn't with Date

That's weird, as this should encode the date properly.

[fischerscode]

I have tried adding a date before signUp using this branch. For me this worked.

ParseUser user = ParseUser("test4", "secret", "test@example.com");
user.set("test", DateTime.now());
user.signUp();

[kwizner]

I have tried adding a date before signUp using this branch. For me this worked.

ParseUser user = ParseUser("test4", "secret", "test@example.com");
user.set("test", DateTime.now());
user.signUp();

Yes, it works well.
To clarify with master branch it will not work so fix is required.

[fischerscode]

I see. Thanks for the clarification.

[fischerscode]

@itmesh I've created a PR #470. Can you please have a quick look at the changes?

[kwizner]

We can close issue, everything is fine, tested on the newest version 1.0.28.