[11.0][mail_activity_board] Add new module that insert activities board in boards. #283

dajuayen · 2018-06-12T16:59:51Z

This module installs the Document module and generates a board with the form, tree, kanban, graph, pivot and calendar views.

It allows to manage all the activities from the same place even though their origin is of different types of objects.

The form view shows a few data of the activity, but it allows to navigate to the object with which it is related.

In the form view of the related object, in the chatter a button has been added that links to the activity board in which the activities of that object are shown.

yajo · 2018-06-15T08:48:20Z

mail_activity_board/README.rst

+Activities Board
+================
+
+This module adds an activity board with form, tree, kanban, calendar, pivot, graph and search views.


Move this paragraph to readme/DESCRIPTION.rst

I have added the readme folder with the file DESCRIPTION.rst

yajo · 2018-06-15T08:48:30Z

mail_activity_board/README.rst

+=============
+
+
+Install it from APPs.


This is implicit, remove it 🔥

yajo · 2018-06-15T08:48:42Z

mail_activity_board/README.rst

+From this smartButton is linked to the activity board, to the view tree,
+which shows the activities related to the opportunity.
+
+From the form view of the activity you can navigate to the origin of the activity.


Move to readm/USAGE.rst

I have added the readme folder with the file USAGE.rst

yajo · 2018-06-15T08:49:00Z

mail_activity_board/README.rst

+* SDI <http://www.sdi.es>
+
+  * David Juaneda Ayensa <djuaneda@sdi.es>
+  * Javier Garcia <jgarcia@sdi.es>


Move to readme/CONTRIBUTORS.rst

I have added the readme folder with the file CONTRIBUTORS.rst

yajo · 2018-06-15T08:49:15Z

mail_activity_board/__init__.py

@@ -0,0 +1,5 @@
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
+# SDI
+# author: djuaneda@sdi.es


I'd remove authors from such a tiny file.

I deleted all comments in the _init_.py files.

yajo · 2018-06-15T09:53:44Z

mail_activity_board/views/mail_activity_view.xml

+                <separator/>
+            </xpath>
+
+            <xpath expr='//search/group' position='inside'>


It would be geat to have a groupby for past, today and future activities (I'm not sure if it exists already)

There are filters for late, today and future activities but if you think it might be interesting to add the groups, it's not hard for me to do it.

It should be enough then, thanks 😉

yajo · 2018-06-15T09:53:59Z

mail_activity_board/views/mail_activity_view.xml

+-->
+
+    <record model="ir.actions.act_window" id="open_boards_activities_form_tree">
+        <field name="name">ACTIVITIES</field>


DO NOT CRY 😆

"Activities"

I will try to overcome it

yajo · 2018-06-15T09:55:03Z

mail_activity_board/views/mail_activity_view.xml

+        <field name="name">ACTIVITIES</field>
+        <field name="res_model">mail.activity</field>
+        <field name="view_type">form</field>
+        <field name="view_mode">tree,form</field>


I think it's better to default to the kanban view.

I've already modified it.

mail_activity_board/views/mail_activity_view.xml

yajo · 2018-06-15T09:56:53Z

mail_activity_board/views/mail_activity_view.xml

+        <field name="name">Boards: My Activities</field>
+        <field name="model_id" ref="model_mail_activity"/>
+        <field name="state">code</field>
+        <field name="code">action = model.action_your_activities()</field>


I think it's generally better the other way around: declare the activity in XML and then load it from code if ever needed (you'll find most times you don't need it).

I've already corrected it.

…improvements in style of code.

dajuayen · 2018-07-09T09:49:54Z

mail_activity_board/views/mail_activity_view.xml

@@ -14,7 +14,7 @@
            <form string="Activity Form" create="false" edit="false" delete="false">
                <sheet string="Activity">
                    <button name="open_origin" type="object" class="centre oe_link" nolabel="1">
-                        <field name="res_name"><field name="res_model_id_name"/></field>
+                        <h1><field name="res_name"/></h1>


Similar to the titles of other forms.

dajuayen · 2018-07-11T09:55:56Z

@gurneyalex @yajo I have managed to pass the tests of travis but I fail to run on runbot.
What is wrong? As much as I searched, I have not been able to find information.
How can I see the logs?

yajo

See the logs here:

Some code comments below. Waiting for runbot to test 😊

yajo · 2018-07-12T06:30:05Z

mail_activity_board/__manifest__.py

-    'license': 'GPL-3',
+    'version': '11.0.1.0.0',
+    'development_status': 'Beta',
+    'category': 'Uncategorized',


Social Network was a good category IMHO.

Changed it.

yajo · 2018-07-12T06:31:39Z

mail_activity_board/models/mail_activity.py

-        default="", related='res_model_id.name',
-        string="Origin", store=False, readonly=True)
+        related='res_model_id.name', string="Origin",
+        store=False, default="", readonly=True)


store=False is the default, you can remove it.

Also, default="" can lead to confusing behavior, because in Odoo, the standard is that empty fields return False always. Having "" instead of False is unconsistent, so please remove that too.

Deleted store=False.
Deleted default="".

yajo · 2018-07-12T06:33:40Z

mail_activity_board/models/mail_activity.py

-        string="Duration", store=False, readonly=True)
-    calendar_event_id_start = fields.Datetime(
-        default=False, related='calendar_event_id.start',
+        related='calendar_event_id.duration', string="Duration",
        store=False, readonly=True)


Most likely, with related='calendar_event_id.duration', readonly=True it should be the same, because all other attributes are inherited from the related field itself.

I have tried to leave the field with only "related" and "readonly" attributes and it has behaved as you have indicated, so I have changed it.

yajo · 2018-07-12T06:33:50Z

mail_activity_board/models/mail_activity.py

        store=False, readonly=True)
+    calendar_event_id_start = fields.Datetime(
+        related='calendar_event_id.start', string="Start",
+        store=False, default=False, readonly=True)


yajo · 2018-07-12T06:33:58Z

mail_activity_board/models/mail_activity.py

        store=False, readonly=True)
+    calendar_event_id_start = fields.Datetime(
+        related='calendar_event_id.start', string="Start",
+        store=False, default=False, readonly=True)
    calendar_event_id_partner_ids = fields.Many2many(
        related='calendar_event_id.partner_ids', string='Attendees',
        store=False, default=False, readonly=True)


yajo · 2018-07-12T06:40:54Z

mail_activity_board/static/src/xml/inherit_chatter.xml

+    <t t-extend="mail.Chatter.Buttons">
+         <t t-jquery="button.o_chatter_button_schedule_activity" t-operation="after">
+            <button t-if="schedule_activity_btn" class="btn btn-sm btn-link o_chatter_button_count_activity" title=" activity">
+                <i class="fa fa-list"/> <t t-if="isMobile">Activities</t><t t-else="">Activities</t>


Why the if?

copy & paste

I already removed it.

Finally I have left it:
<t t-if = "isMobile"> List </ t> Activities list </ t>

because it shows a different legend if 'isMobile' or not.

yajo · 2018-07-12T06:42:14Z

mail_activity_board/views/mail_activity_view.xml

+
+                                <div class="o_kanban_record_bottom">
+                                    <div class="oe_kanban_bottom_left">
+                                        <t t-if="record.date_deadline.raw_value and record.date_deadline.raw_value lt (new Date())">


If record.date_deadline
and
its value is previous previous to today
add the class "text-danger".

Is it badly used?

There's still this. lt is nothing. Did you mean to use <? Also, I think you're comparing a String with a Date, which will very likely produce false positives (not sure, just guessing...)

Answering to #283 (comment) here to avoid losing context.

The fact that you copy-pasted code doesn't mean you copied good code. 😜

Normally, codecov won't detect this, because the qweb view you write in a kanban is actually a client-side (JS) qweb one. This means that it's not executed when installing, but when you browse it.

See this screenshot from Firefox JS console, where you can see that you're producing a syntax error, and a false positive:

@yajo I think it is ready.
The field 'date_deadline' in the class 'mail.activity' is required, so I have removed the check if it is empty.

yajo · 2018-07-12T06:42:46Z

mail_activity_board/views/mail_activity_view.xml

+                                                    <field name="date_deadline"/>
+                                                </span>
+                                            </t>
+                                            <t t-if="record.activity_category.raw_value=='meeting'">


Can't you use t-else?

No problem.

yajo · 2018-07-12T06:43:01Z

mail_activity_board/views/mail_activity_view.xml

+                                                    <field name="date_deadline"/>
+                                                </span>
+                                            </t>
+                                            <t t-if="record.activity_category.raw_value=='meeting'">


yajo · 2018-07-12T06:43:43Z

mail_activity_board/views/mail_activity_view.xml

+                <separator/>
+            </xpath>
+
+            <xpath expr='//search/group' position='inside'>


It should be enough then, thanks 😉

elicoidal · 2018-07-13T07:35:57Z

Runbot seems not available?

gurneyalex · 2018-07-15T09:37:18Z

If you check runbot status, it is "orange" which means that there are some warnings in the logs.

Checking the logs show:

2018-07-13 12:43:26,081 165 INFO openerp_test odoo.modules.loading: loading mail_activity_board/views/mail_activity_view.xml
2018-07-13 12:43:26,110 165 WARNING openerp_test odoo.models: ir.ui.view.create() includes unknown fields: domain
2018-07-13 12:43:26,135 165 WARNING openerp_test odoo.models: ir.ui.view.create() includes unknown fields: context

gurneyalex · 2018-07-15T19:24:26Z

mail_activity_board/views/mail_activity_view.xml

+        <field name="name">mail.activity.boards.view.tree</field>
+        <field name="model">mail.activity</field>
+        <field name="inherit_id" ref="mail.mail_activity_view_tree"/>
+        <field name="domain">[]</field>


this line is causing the build failure on runbot

Thank you for answering the question.

gurneyalex · 2018-07-15T19:24:49Z

mail_activity_board/views/mail_activity_view.xml

+         <field name="name">mail.activity.boards.view.kanban</field>
+         <field name="model">mail.activity</field>
+         <field name="priority" eval="10"/>
+         <field name="context">{}</field>


this line is causing a red bulid on runbot

Thanks again.

yajo · 2018-07-16T07:26:31Z

Fix runbot:

2018-07-13 12:43:26,110 165 WARNING openerp_test odoo.models: ir.ui.view.create() includes unknown fields: domain
2018-07-13 12:43:26,135 165 WARNING openerp_test odoo.models: ir.ui.view.create() includes unknown fields: context

…attributes in views.

dajuayen · 2018-07-17T00:23:41Z

Hi @yajo.
Runbot test pass, but now something called 'codecov' appeared.
What do I have to do now to finish the push request?

yajo

You need to simplify that JS, friend 😉

yajo · 2018-07-17T07:10:20Z

mail_activity_board/models/mail_activity.py

-        related='calendar_event_id.partner_ids', string='Attendees',
-        default=False, readonly=True)
+        related='calendar_event_id.partner_ids',
+        Stringreadonly=True)


Oops, wrong parameter.

yajo · 2018-07-17T07:11:51Z

mail_activity_board/static/src/js/override_chatter.js

+            } else if (count === 1){
+                str = _t('One activity');
+            } else {
+                str = ''+count+' '+_t('activities');


This method makes sense in your language and some others, but did you know that Russian has different plurals, depending on if the number ends by 2-4 or by other number? And of course, they have singular too. And it's not a corner case. You likely don't want this kind of methods that return different strings depending on the number. Gettext itself should handle that, and if it doesn't, it's upstream's problem (and also I don't think it will affect you, since your language follows the same rules as English).

It's much simpler to be dumber and just do:

return _.str.sprintf(_t("%d activities"));

... which wouldn't be so extremely bad if rendered as 1 activities. You can even translate that into Spanish as %d actividad/es, which is perfectly OK.

This also should make you realize that using a method just for this is not much needed. You can just do that call inline and it should be OK.

In any case, this is being used just to add a title attribute to the button, but we can simply hardcode a more useful title and forget about all of this; so, just remove this whole _formatActivitiesList method please.

I'm just trying to give you a bigger picture about how translations work (or should work), so you can remove all this complexities from your mind (and code!). 😉

Thanks for advice.
When I don't know how to make something I use to find in ODOO folder a piece of code similar to I want to do and copy it.
I found in followers.js something similar:

_formatFollowers: function (count){ var str = ''; if (count <= 0) { str = _t('No follower'); } else if (count === 1){ str = _t('One follower'); } else { str = ''+count+' '+_t('followers'); } return str; }

In next times I'll try to be more efficient and follow your wise advice.

I 🔥 it already

yajo · 2018-07-17T07:48:56Z

mail_activity_board/static/src/xml/inherit_chatter.xml

+    <t t-extend="mail.Chatter.Buttons">
+         <t t-jquery="button.o_chatter_button_schedule_activity" t-operation="after">
+            <button t-if="schedule_activity_btn" class="btn btn-sm btn-link o_chatter_button_list_activity">
+                 <span class="o_list_activity label label-primary"/> <t t-if="isMobile">List</t><t t-else="">Activities list</t>


Some points here:

If you need to display something depending on screen size, it's better to do it this way by pure bootstrap CSS:

<span class="hidden-xs">Activities list</span> <span class="visible-xs-inline">Activities</span>

The less JS and the more CSS you use, the faster everything is. However...

... KISS; just use "Activities". The "list" word here adds nothing useful really.

Additions should be done here, and not above in the _render() method; however, it's true that will only be possible after [IMP] web: Expose chatter widget to render buttons odoo/odoo#25801 is merged.

So, it should end up being similar to this, as of today:

<button t-if="schedule_activity_btn" class="btn btn-sm btn-link o_chatter_button_list_activity" title="See activities list" type="button"> Activities </button>

Once odoo/odoo#25801 is merged, if ever, it could become this:

<button t-if="schedule_activity_btn" class="btn btn-sm btn-link o_chatter_button_list_activity" title="See activities list" type="button"> + <span class="o_list_activity label label-primary"> + <t t-esc="widget.fields.activity.activities.length"/> + </span> Activities </button>

As I have already mentioned to you, when I have to do something, I try to look at how it is done in another part of the code and imitate it.

I thought that instead of using an icon, I could use the number of activities for it, in the style of followers and the number of activities when the list of activities is minimized.

What a pity
😔

yajo · 2018-07-17T07:55:51Z

mail_activity_board/static/src/js/override_chatter.js

+                if (self.fields.activity) {
+                    self.$('.o_list_activity')
+                                .html(self.fields.activity.activities.length)
+                                .parent().attr("title", self._formatActivitiesList(self.fields.activity.activities.length));


I see that you're duplicating almost all of upstream method just to add these 2 lines, breaking the call chain. Sorry, but that shouldn't be done. 🚫

You have 3 options:

Override the start() method, call its _super(), and, once done, make the changes into this.$topbar, which is there rendered synchronously. All of this deferred rendering of fields has no relation with what you're modifying, which is actually the buttons bar, not the fields themselves.

Vote for [IMP] web: Expose chatter widget to render buttons odoo/odoo#25801 to be merged, and just do all overrides directly in Qweb, as explained below.

Simply show static buttons. That can be done through Qweb only, and wouldn't need [IMP] web: Expose chatter widget to render buttons odoo/odoo#25801 to be merged. I recommend this approach, at least for now; see below.

Remove this _render() method in any case.

I removed _render().

But I wrote down option 1. I'm stubborn.

yajo · 2018-07-17T07:57:43Z

mail_activity_board/static/src/js/override_chatter.js

+        }),
+
+        _onListActivity: function (event) {
+            event.preventDefault();


Not needed if you add type="button" to the button. Remove and see comments below.

You mean only the line:

event.preventDefault();

yajo

Much better, getting closer 😉

yajo · 2018-07-19T07:53:54Z

mail_activity_board/views/mail_activity_view.xml

+
+                                <div class="o_kanban_record_bottom">
+                                    <div class="oe_kanban_bottom_left">
+                                        <t t-if="record.date_deadline.raw_value and record.date_deadline.raw_value lt (new Date())">


There's still this. lt is nothing. Did you mean to use <? Also, I think you're comparing a String with a Date, which will very likely produce false positives (not sure, just guessing...)

dajuayen · 2018-09-16T18:24:59Z

@yajo I have added a change to the form view on the activity board for those that are meeting type. Check them out to see what you think.
The other problems are solved.

mail_activity_board/views/mail_activity_view.xml

LoisRForgeFlow

Tested on runbot and functionally looks great! 👍

Thanks!

dajuayen · 2018-09-20T10:21:00Z

@lreficent Thaks you!
We have developed a module that keeps mail.activiy done and when we had this fact we were going to upload it but I have seen that you have advanced. This PR was my first, between that and that I have not had time to dedicate, it has been very long.
The other day I downloaded your code and I was testing it join to the mail_activity_board and so you can see together they work perfectly.
Anyway I made a comment in the PR to see what you think.
What do you think what I said? How do you carry the PR?

dajuayen · 2018-11-02T08:27:36Z

@yajo Why not just upload the PR to Social? Is something missing to be done so that it can be uploaded to the repository?

yajo · 2018-11-02T09:38:01Z

Sorry I just forgot this one. Merging.

…d in boards. (OCA#283) * [ADD] Module that insert activities board. * [FIX] Author error in __manifest__ file and style changes. * [FIX] Fix replace in view, rename files and style changes. * [FIX] Enumerated list ends without a blank line; unexpected unindent. * [FIX] Name fail. * [FIX] Bug in view. * [FIX] Add button Activities in mail.thread and readme folder. Others improvements in style of code. * [FIX] Type 'tree' not found in registry: problem solved. * [FIX] Dependence change: 'mail' for 'calendar'. * [FIX] Eliminated unnecessary imports. * [FIX] Bugs about js and if/else. * [FIX] Improvements following guide lines and eliminating unnecessary attributes in views. * [ADD] Added counter in the 'Activities List' button. * [FIX] Bugs in javascript with 'Activities' button. * [ADD] Tests folder. * [FIX] Deleted references to modules not installed. * [FIX] Formatting javascript. * [FIX] Bug: added a soft line before a class. * [FIX] Bug: OCA#283 (comment) * [FIX] Escaping 'lt' in xml file. Bug: OCA#283 (comment) * [FIX] The meeting attendees are shown in kanban mode on the meeting board. * [FIX] Hide in form view of the activity board the assistant field if the activity is not a meeting type or if there are no assistants. * [FIX] Change to default kanban view for partners.

Syncing from upstream OCA/social (11.0)

…d in boards. (#283) * [ADD] Module that insert activities board. * [FIX] Author error in __manifest__ file and style changes. * [FIX] Fix replace in view, rename files and style changes. * [FIX] Enumerated list ends without a blank line; unexpected unindent. * [FIX] Name fail. * [FIX] Bug in view. * [FIX] Add button Activities in mail.thread and readme folder. Others improvements in style of code. * [FIX] Type 'tree' not found in registry: problem solved. * [FIX] Dependence change: 'mail' for 'calendar'. * [FIX] Eliminated unnecessary imports. * [FIX] Bugs about js and if/else. * [FIX] Improvements following guide lines and eliminating unnecessary attributes in views. * [ADD] Added counter in the 'Activities List' button. * [FIX] Bugs in javascript with 'Activities' button. * [ADD] Tests folder. * [FIX] Deleted references to modules not installed. * [FIX] Formatting javascript. * [FIX] Bug: added a soft line before a class. * [FIX] Bug: OCA/social#283 (comment) * [FIX] Escaping 'lt' in xml file. Bug: OCA/social#283 (comment) * [FIX] The meeting attendees are shown in kanban mode on the meeting board. * [FIX] Hide in form view of the activity board the assistant field if the activity is not a meeting type or if there are no assistants. * [FIX] Change to default kanban view for partners.

djuaneda.sdi added 6 commits June 12, 2018 17:10

[ADD] Module that insert activities board.

d56c2f0

[FIX] Author error in __manifest__ file and style changes.

4b929c4

[FIX] Fix replace in view, rename files and style changes.

7ed0a4c

[FIX] Enumerated list ends without a blank line; unexpected unindent.

919f33e

[FIX] Name fail.

cc486cb

[FIX] Bug in view.

4caf09e

yajo added the needs review label Jun 15, 2018

yajo modified the milestones: 10.0, 11.0 Jun 15, 2018

yajo requested changes Jun 15, 2018

View reviewed changes

david.juaneda and others added 3 commits July 8, 2018 13:49

[FIX] Add button Activities in mail.thread and readme folder. Others …

ad7b784

…improvements in style of code.

[FIX] Type 'tree' not found in registry: problem solved.

f870bad

[FIX] Dependence change: 'mail' for 'calendar'.

ce43235

dajuayen commented Jul 9, 2018

View reviewed changes

[FIX] Eliminated unnecessary imports.

82dd15e

yajo requested changes Jul 12, 2018

View reviewed changes

[FIX] Bugs about js and if/else.

470b3dc

gurneyalex reviewed Jul 15, 2018

View reviewed changes

david.juaneda added 2 commits July 17, 2018 00:36

[FIX] Improvements following guide lines and eliminating unnecessary …

4456f9e

…attributes in views.

[ADD] Added counter in the 'Activities List' button.

e93ca79

yajo mentioned this pull request Jul 17, 2018

[IMP] web: Expose chatter widget to render buttons odoo/odoo#25801

Closed

yajo requested changes Jul 17, 2018

View reviewed changes

[FIX] Bugs in javascript with 'Activities' button.

676a3ed

yajo requested changes Jul 19, 2018

View reviewed changes

yajo requested changes Sep 17, 2018

View reviewed changes

[FIX] Change to default kanban view for partners.

97297af

yajo approved these changes Sep 18, 2018

View reviewed changes

LoisRForgeFlow approved these changes Sep 20, 2018

View reviewed changes

yajo merged commit c59f6ff into OCA:11.0 Nov 2, 2018

SiesslPhillip pushed a commit to grueneerde/OCA-social that referenced this pull request Nov 20, 2024

Merge pull request OCA#283 from OCA/11.0

447af94

Syncing from upstream OCA/social (11.0)

		=============


		Install it from APPs.

[11.0][mail_activity_board] Add new module that insert activities board in boards. #283

[11.0][mail_activity_board] Add new module that insert activities board in boards. #283

Conversation

dajuayen commented Jun 12, 2018 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

dajuayen Jul 8, 2018 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

dajuayen Jul 8, 2018 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

dajuayen commented Jul 11, 2018

yajo left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

dajuayen Jul 16, 2018 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

elicoidal commented Jul 13, 2018

gurneyalex commented Jul 15, 2018

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

yajo commented Jul 16, 2018

dajuayen commented Jul 17, 2018

yajo left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

dajuayen Jul 17, 2018 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

dajuayen Jul 17, 2018 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

yajo left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

dajuayen commented Jun 12, 2018 •

edited

Loading

dajuayen Jul 8, 2018 •

edited

Loading

dajuayen Jul 8, 2018 •

edited

Loading

dajuayen Jul 16, 2018 •

edited

Loading

dajuayen Jul 17, 2018 •

edited

Loading

dajuayen Jul 17, 2018 •

edited

Loading