Support Relation.

[yulingtianxia]

Now you can use ParseRelation in flutter_parse_sdk.
Resolve this issue: #169

[phillwiggins]

Amazing work! Thank you very much for your contribution.

[phillwiggins]

Just a quick one. We have some changes that are on branch v.1.0.22. Is it possible for you to create a merge request to branch 1.0.22 and see if we can merge it there?

[phillwiggins]

Thank you. Can you look at the merge conflicts please?

[phillwiggins]

Brilliant. Thanks!

[yulingtianxia]

I think this merge make some mistakes. Because it's merge from master to v.1.0.22, some change shouldn't be there.

[phillwiggins]

Which branch is affected by this? Master or v1.0.22?

…

On Fri, 31 May 2019 at 14:37, 杨萧玉 ***@***.***> wrote: I think this merge make some mistakes. Because it's merge from master to v.1.0.22, some change shouldn't be there. — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub <#178>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AB4CPXVN3OUPMFD4DITJM2LPYESZBANCNFSM4HRW2ZLA> .

-- Kind Regards Phill Wiggins [email protected]

[yulingtianxia]

Which branch is affected by this? Master or v1.0.22?
…
On Fri, 31 May 2019 at 14:37, 杨萧玉 @.***> wrote: I think this merge make some mistakes. Because it's merge from master to v.1.0.22, some change shouldn't be there. — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub <#178?email_source=notifications&email_token=AB4CPXUUJMWC37W7WIUG5XLPYESZBA5CNFSM4HRW2ZLKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWVHSYY#issuecomment-497711459>, or mute the thread https://github.com/notifications/unsubscribe-auth/AB4CPXVN3OUPMFD4DITJM2LPYESZBANCNFSM4HRW2ZLA .
-- Kind Regards Phill Wiggins [email protected]

Both.

[phillwiggins]

Are you able to fix V1.0.22 branch and I will revert master?

…

On Fri, 31 May 2019 at 14:51, 杨萧玉 ***@***.***> wrote: Which branch is affected by this? Master or v1.0.22? … <#m_-1496923601104661568_> On Fri, 31 May 2019 at 14:37, 杨萧玉 *@*.***> wrote: I think this merge make some mistakes. Because it's merge from master to v.1.0.22, some change shouldn't be there. — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub <#178 <#178>, or mute the thread https://github.com/notifications/unsubscribe-auth/AB4CPXVN3OUPMFD4DITJM2LPYESZBANCNFSM4HRW2ZLA . -- Kind Regards Phill Wiggins ***@***.*** Both. — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub <#178>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AB4CPXVCNC2SVOLAKHE6H2DPYEUP7ANCNFSM4HRW2ZLA> .

-- Kind Regards Phill Wiggins [email protected]

[yulingtianxia]

Are you able to fix V1.0.22 branch and I will revert master?
…

OK.

[phillwiggins]

Thank you

…

On Fri, 31 May 2019 at 15:45, 杨萧玉 ***@***.***> wrote: Are you able to fix V1.0.22 branch and I will revert master? … <#m_-2908689569781745733_> OK. — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub <#178>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AB4CPXSN2L2X6PFJJEPPG5LPYE2W7ANCNFSM4HRW2ZLA> .

-- Kind Regards Phill Wiggins [email protected]

[sidiabale]

Hi @phillwiggins and @yulingtianxia , I just stumbled on this PR. Incidentally, I needed to query existing relations and was playing around with an implementation before seeing this PR. It appears this aspect is not implemented here (can someone confirm that?).

I'm referring to the $relatedTo query explained here. The implementation is pretty straightforward (in parse_query.dart)

/// Retrieves related objets where [String] column is a relation field to the class [String] className
  void whereRelatedTo(String column, String className, String objectId) {
    queries.add(MapEntry<String, dynamic>(
        _SINGLE_QUERY, '\"\$relatedTo\":{\"object\":{\"__type\":\"Pointer\",\"className\":\"$className\",\"objectId\":\"$objectId\"},\"key\":\"$column\"}'));
  }

Can we add that to the ongoing implementation? I can make a PR if needed.

@phillwiggins By the way, are there any regression tests for this lib? Pretty handy as I've seen regression in Parse Server in the past and it does help to detect that a.s.a.p.

[yulingtianxia]

@sidiabale Amazing work! Your work does not conflict with this PR. This PR makes it possible to add or remove relations for some Class. You add the query API.

[sidiabale]

Great. What branch should I make a PR to? Op ma 3 jun. 2019 07:16 schreef 杨萧玉 <[email protected]>:

…

@sidiabale <https://github.com/sidiabale> Amazing work! Your work does not conflict with this PR. This PR makes it possible to add or remove relations for some Class. You add the query API. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#178>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAJEA5D7KM26UDKLQZRKWNLPYSSLVANCNFSM4HRW2ZLA> .

[phillwiggins]

Release v1.0.22 please!

…

On Mon, Jun 3, 2019, 17:39 Chidiebere Okwudire ***@***.***> wrote: Great. What branch should I make a PR to? Op ma 3 jun. 2019 07:16 schreef 杨萧玉 ***@***.***>: > @sidiabale <https://github.com/sidiabale> Amazing work! Your work does > not conflict with this PR. This PR makes it possible to add or remove > relations for some Class. You add the query API. > > — > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub > < #178 >, > or mute the thread > < https://github.com/notifications/unsubscribe-auth/AAJEA5D7KM26UDKLQZRKWNLPYSSLVANCNFSM4HRW2ZLA > > . > — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#178>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AB4CPXVFLMRWKSHMFMEPX53PYVCKRANCNFSM4HRW2ZLA> .

[sidiabale]

@phillwiggins Done (see PR 192)