*** mjsir911 has joined #novawebdev | 08:45 | |
*** jelkner has joined #novawebdev | 09:31 | |
*** dsmall has joined #novawebdev | 09:32 | |
jelkner | Good morning, dsmall! | 09:32 |
---|---|---|
dsmall | morning jelkner | 09:32 |
*** replaceafill has joined #novawebdev | 10:06 | |
replaceafill | good morning jelkner | 10:06 |
replaceafill | good morning dsmall | 10:06 |
dsmall | good morning replaceafill | 10:07 |
replaceafill | dsmall, have time for discussing your blog view? | 10:12 |
dsmall | replaceafill, sure | 10:12 |
replaceafill | dsmall, https://gitlab.com/duncan-small/Tendenci-Blog/blob/master/views.py#L8-9 | 10:12 |
replaceafill | dsmall, categories_list contains all the categories in the system, correct | 10:13 |
replaceafill | dsmall, sorted by name | 10:13 |
dsmall | replaceafill, correct | 10:13 |
replaceafill | dsmall, what's the purpose of big_list? | 10:14 |
dsmall | replaceafill, it keeps a ledger of the number of posts for each category | 10:14 |
replaceafill | dsmall, have you seen this method in the queryset api: https://docs.djangoproject.com/en/1.8/ref/models/querysets/#count | 10:15 |
replaceafill | dsmall, i was wondering if we could avoid the loop by using that | 10:16 |
dsmall | replaceafill, It would get rid of one loop | 10:16 |
replaceafill | dsmall, the inner one? | 10:17 |
dsmall | replaceafill, yes | 10:17 |
replaceafill | dsmall, right | 10:17 |
dsmall | replaceafill, I can implement that today | 10:18 |
replaceafill | dsmall, sure, no rush | 10:18 |
replaceafill | dsmall, i remember there's a pattern for that | 10:19 |
replaceafill | dsmall, i mean for that kind of nested loop | 10:19 |
replaceafill | dsmall, but as usual i can't find the doc... | 10:19 |
replaceafill | dsmall, oh right | 10:19 |
replaceafill | dsmall, https://docs.djangoproject.com/en/1.8/ref/models/querysets/#annotate | 10:19 |
dsmall | replaceafill, oh cool | 10:19 |
replaceafill | dsmall, have you used the django shell? | 10:19 |
replaceafill | dsmall, python manage.py shell? | 10:20 |
dsmall | replaceafill, not yet, but I know how | 10:20 |
replaceafill | dsmall, ah ok | 10:20 |
replaceafill | dsmall, could you give the annotate approach a try | 10:20 |
replaceafill | dsmall, if you have other things to do, that's fine | 10:20 |
replaceafill | dsmall, we can optimize later | 10:20 |
replaceafill | dsmall, i just want this on your radar | 10:20 |
dsmall | replaceafill, im currently messing around with comments, but I havent gotten very far with that | 10:21 |
replaceafill | dsmall, ah ok | 10:21 |
replaceafill | dsmall, do you have any questions about that? | 10:21 |
replaceafill | dsmall, i haven't really explored those comments packages yet | 10:22 |
replaceafill | dsmall, were you able to figure out the templates involved in the rendering you want to modify? | 10:22 |
dsmall | replaceafill, I don't have any specific questions, just general confusion. It is kinda hard because most of the packages are for django 2.0 and the documentation for it is for django 2.0 | 10:23 |
replaceafill | dsmall, ah | 10:23 |
dsmall | replaceafill, I found the template, but I havent be able to successfully modify it | 10:23 |
replaceafill | dsmall, ok | 10:23 |
dsmall | replaceafill, could you help me with a urlpattern really quick? | 11:28 |
replaceafill | dsmall, sure | 11:28 |
dsmall | replaceafill, https://paste.pound-python.org/show/YZRcL05YeGAtxUnqAmok/ | 11:28 |
dsmall | replaceafill, thats what the docs for the package said to put into urls.py but i dont think it will work | 11:29 |
replaceafill | dsmall, this package: https://github.com/danirus/django-comments-xtd ? | 11:30 |
dsmall | replaceafill, correct | 11:30 |
replaceafill | dsmall, you're here: https://django-comments-xtd.readthedocs.io/en/latest/quickstart.html ? | 11:30 |
dsmall | replaceafill, Yup, step 8 | 11:30 |
replaceafill | dsmall, and i assume you're adding that in your tendenci project in conf.local_urls? | 11:31 |
replaceafill | conf.local_urls.extrapatterns? | 11:31 |
dsmall | replaceafill, I was ust doing conf/urls.py | 11:31 |
dsmall | just** | 11:31 |
dsmall | replaceafill, wouldn't the 'include' part be problematic? | 11:32 |
replaceafill | dsmall, the effect should be the same but it's better to use local_urls | 11:32 |
dsmall | replaceafill, ok | 11:32 |
replaceafill | dsmall, why do you think it's a problem? | 11:32 |
replaceafill | dsmall, the only thing i note is the lack of $ at the end of the regexp | 11:32 |
replaceafill | dsmall, but that's not a big deal | 11:32 |
dsmall | replaceafill, nevermind, in my conf.urls.py none of them had 'include' | 11:33 |
replaceafill | dsmall, ah | 11:33 |
replaceafill | dsmall, but they do in conf.local_urls, right? | 11:33 |
dsmall | replaceafill, Yes | 11:33 |
replaceafill | dsmall, got it | 11:33 |
dsmall | replaceafill, something broke :/ | 11:34 |
replaceafill | dsmall, paste error? | 11:35 |
dsmall | replaceafill, https://paste.pound-python.org/show/oAdCZSpfwZuE7hwMVcKa/ | 11:35 |
replaceafill | dsmall, ah yes that's usually a misconfiguration in the urls | 11:35 |
dsmall | replaceafill, oops | 11:36 |
replaceafill | dsmall, paste conf.local_urls? | 11:36 |
replaceafill | dsmall, and conf.urls? | 11:36 |
dsmall | replaceafill, local_urls: https://paste.pound-python.org/show/wRTcs2JlxxkPdXLmhVOx/ | 11:36 |
dsmall | replaceafill, urls: https://paste.pound-python.org/show/Qi9pDXShJhbDSfmTzhlX/ | 11:37 |
replaceafill | dsmall, paste conf.local_settings? | 11:38 |
replaceafill | dsmall, did the migrate command in step 7 worked? | 11:39 |
dsmall | replaceafill, conf.local_settings: https://paste.pound-python.org/show/n6f4MUVpLVXcjVtGApsK/ | 11:39 |
dsmall | replaceafill, Ya it worked fine | 11:39 |
dsmall | replaceafill, I even ran makemigrations before | 11:40 |
dsmall | replaceafill, I don't have django.contrib.sites in my INSTALLED_APPS, bc when i did have it there it told me: "django.core.exceptions.ImproperlyConfigured: Application labels aren't unique, duplicates: sites" | 11:42 |
replaceafill | dsmall, ah yes, that's in tendenci settings.py | 11:42 |
replaceafill | dsmall, https://github.com/tendenci/tendenci/blob/master/tendenci/settings.py | 11:43 |
dsmall | replaceafill, should I move it? | 11:43 |
replaceafill | dsmall, i don't think you need to | 11:43 |
replaceafill | dsmall, the only order requirement i see is Add 'django_comments_xtd' and 'django_comments', in that order | 11:43 |
dsmall | replaceafill, right | 11:44 |
replaceafill | dsmall, you could try putting those at the end of INSTALLED_APPS | 11:44 |
dsmall | replaceafill, Okay | 11:44 |
replaceafill | dsmall, it could be a django 2 problem | 11:46 |
replaceafill | dsmall, run "pip freeze | grep comment" in your virtualenv | 11:47 |
dsmall | replaceafill, it still doesn't work, so it could | 11:47 |
dsmall | replaceafill, okay | 11:47 |
replaceafill | dsmall, to see what version you got installed | 11:47 |
dsmall | replaceafill, "django-comments-xtd==2.1.0, django-contrib-comments==1.8.0" | 11:48 |
replaceafill | dsmall, as far as i understand the package should be compatible with 1.8 and 2.0 | 11:50 |
dsmall | replaceafill, right, in the readme it said it supports 1.8 | 11:51 |
dsmall | replaceafill, (and python 2.7) | 11:51 |
replaceafill | dsmall, right it has the usual "look here first if not look here": https://github.com/danirus/django-comments-xtd/blob/master/django_comments_xtd/views.py#L12-L15 | 11:52 |
dsmall | replaceafill, hmmmm | 11:52 |
replaceafill | dsmall, so the dumb way i debug url issues is | 11:53 |
replaceafill | dsmall, i insert a line like this in the conf.local_urls module | 11:54 |
replaceafill | import pdb; pdb.set_trace(); | 11:54 |
replaceafill | dsmall, above extrapatterns | 11:54 |
dsmall | replaceafill, okay, i'll put that in | 11:54 |
replaceafill | dsmall, then start the server | 11:54 |
replaceafill | dsmall, and see if you get a (pdb) prompt | 11:54 |
replaceafill | dsmall, if not, try visiting a page | 11:55 |
replaceafill | dsmall, you should get the (pdb) prompt either way | 11:55 |
dsmall | replaceafill, got one | 11:55 |
replaceafill | dsmall, | 11:55 |
replaceafill | ok | 11:55 |
replaceafill | dsmall, now try this: | 11:55 |
replaceafill | include('django_comments_xtd.urls') | 11:55 |
replaceafill | dsmall, and see if you get an exception | 11:55 |
dsmall | replaceafill, "ImportError: No module named urls | 11:57 |
dsmall | " | 11:57 |
replaceafill | dsmall, type "q" | 11:59 |
dsmall | replaceafill, ok | 11:59 |
replaceafill | dsmall, to get out of the prompt | 11:59 |
replaceafill | dsmall, and stop the server | 11:59 |
replaceafill | dsmall, get into the django shell | 11:59 |
replaceafill | dsmall, "python manage.py shell" | 11:59 |
replaceafill | dsmall, i'm not sure if that requires urls working | 11:59 |
replaceafill | dsmall, let me test locally | 12:00 |
dsmall | replaceafill, https://paste.pound-python.org/show/4U59XzwQhcxUhz2E6Z0V/ | 12:00 |
dsmall | replaceafill, it wouldn't let me go into the shell | 12:00 |
replaceafill | dsmall, you can remove the pdb line in conf.local_urls | 12:01 |
dsmall | replaceafill, ok | 12:01 |
replaceafill | dsmall, did you keep attendance at the beginning of INSTALLED_APPS? | 12:02 |
dsmall | replaceafill, ya I didn't change that | 12:02 |
replaceafill | dsmall, let me try adding the package to one of my instances | 12:02 |
dsmall | replaceafill, Okay | 12:02 |
replaceafill | migrating.... | 12:06 |
replaceafill | dsmall, hhmm it worked, no errors | 12:09 |
dsmall | replaceafill, weird | 12:10 |
replaceafill | dsmall, do you have any changes in your blog package? | 12:10 |
dsmall | replaceafill, my repo should be updated | 12:11 |
dsmall | replaceafill, it is up to date | 12:12 |
replaceafill | dsmall, and if you comment out the comments url in conf.local_urls everything works? | 12:12 |
dsmall | replaceafill, I'll find out | 12:13 |
replaceafill | dsmall, after enabling the package | 12:13 |
replaceafill | dsmall, do you know what should one be looking at next? | 12:13 |
dsmall | replaceafill, everything is working again | 12:13 |
replaceafill | dsmall, i mean, any page or view to visit | 12:13 |
dsmall | replaceafill, huh? | 12:14 |
replaceafill | dsmall, i guess one has to follow this tutorial: https://django-comments-xtd.readthedocs.io/en/latest/tutorial.html#ref-tutorial | 12:14 |
replaceafill | dsmall, to learn how to use the features in the comments package | 12:14 |
dsmall | replaceafill, ahh | 12:14 |
replaceafill | dsmall, the admin seems to work for me | 12:16 |
replaceafill | dsmall, could you zip your conf directory and send it to me | 12:17 |
dsmall | replaceafill, sure | 12:17 |
replaceafill | dsmall, i just want to compare | 12:17 |
replaceafill | dsmall, if you could zip the whole project directory and it's not too big | 12:18 |
replaceafill | dsmall, that'll be better | 12:18 |
dsmall | replaceafill, sure | 12:18 |
replaceafill | dsmall, i assume your project directory doesn't contain the virtualenv | 12:18 |
dsmall | corrects | 12:19 |
*** mjsir911 has joined #novawebdev | 12:20 | |
replaceafill | dsmall, oh! | 12:21 |
replaceafill | dsmall, it may be the theme | 12:21 |
dsmall | replaceafill, really? | 12:21 |
replaceafill | dsmall, i just realized you have a theme branch | 12:21 |
replaceafill | dsmall, are you using that? | 12:21 |
replaceafill | dsmall, when i switched to it i got the error | 12:21 |
dsmall | replaceafill, ohhh | 12:21 |
dsmall | replaceafill, i dont think ive changed anything from the ora_dev branch | 12:22 |
replaceafill | dsmall, no, but i was using master | 12:22 |
dsmall | replaceafill, oh okay | 12:22 |
dsmall | replaceafill, I'll try that | 12:22 |
dsmall | replaceafill, still isn't working | 12:24 |
replaceafill | dsmall, now i even have your blog package enabled | 12:30 |
replaceafill | dsmall, ah nice, i see the error now | 12:30 |
replaceafill | dsmall, ah | 12:32 |
replaceafill | dsmall, https://paste.pound-python.org/show/BMlv1MVqJPtMgraEcik5/ | 12:32 |
replaceafill | dsmall, it's not the comments package version | 12:32 |
replaceafill | dsmall, it's the rest framework package | 12:32 |
replaceafill | dsmall, which is a dependency | 12:32 |
dsmall | replaceafill, ohh okay | 12:33 |
dsmall | replaceafill, that error looks like mine | 12:33 |
replaceafill | dsmall, https://paste.pound-python.org/show/BMlv1MVqJPtMgraEcik5/ | 12:35 |
replaceafill | oops | 12:35 |
replaceafill | dsmall, https://github.com/encode/django-rest-framework/issues?q=milestone%3A%223.7.0+Release%22 | 12:35 |
replaceafill | "Remove Django 1.8 & 1.9 compatibility code" | 12:35 |
replaceafill | dsmall, we need something lower that rest framework 3.7 | 12:35 |
replaceafill | dsmall, http://www.django-rest-framework.org/topics/release-notes/ | 12:35 |
replaceafill | dsmall, let me try 3.6.4 | 12:36 |
*** mr_german has joined #novawebdev | 12:36 | |
mr_german | good morning jelkner | 12:36 |
dsmall | replaceafill, alright, do you still want my project dir? | 12:36 |
replaceafill | dsmall, no | 12:36 |
dsmall | replaceafill, ok | 12:36 |
replaceafill | dsmall, thanks i was able to set everything up here | 12:36 |
dsmall | replaceafill, np, im going to grab lunch, be back in <1 hour | 12:37 |
replaceafill | dsmall, sure | 12:37 |
replaceafill | dsmall, pip install djangorestframework==3.6.4 | 12:38 |
replaceafill | dsmall, try that ^ and let me know | 12:38 |
mr_german | jelkner, I couldn't find any website specs for ormd | 12:40 |
jelkner | mr_german, did you see the email? | 13:19 |
mr_german | jelkner, yeah, but I can't any doc with the website specs | 13:20 |
jelkner | mr_german, i don't understand. | 13:20 |
jelkner | did you mean you can't find any doc? | 13:21 |
jelkner | hold on... | 13:21 |
jelkner | https://docs.google.com/document/d/1aQupGYCIplRIqd_8csfrJ2BKYJjyMmwJP_QkSy1OcCg/edit?ts=5b0ef152 | 13:21 |
jelkner | can you see this document, mr_german? | 13:22 |
mr_german | jelkner, yes! | 13:22 |
jelkner | replaceafill, i don't know the person whose email is bouncing | 13:22 |
jelkner | excellent, mr_german | 13:23 |
replaceafill | jelkner, cool, nothing we can do then | 13:23 |
mr_german | jelkner, thanks! | 13:23 |
jelkner | please see how much of this you can do by Sunday | 13:23 |
jelkner | they are coming in to learn to use the system | 13:23 |
jelkner | btw. replaceafill how are we on funding for mr_german? | 13:23 |
jelkner | is the well dry? | 13:23 |
replaceafill | jelkner, dry | 13:23 |
replaceafill | jelkner, yep | 13:23 |
jelkner | ouch | 13:23 |
jelkner | i need to send you the 3K | 13:24 |
replaceafill | jelkner, want to see the spreadsheet? | 13:24 |
jelkner | all i need is a number | 13:24 |
jelkner | but let me check your transfer first | 13:24 |
replaceafill | jelkner, $5M | 13:24 |
replaceafill | jelkner, :D | 13:24 |
replaceafill | jelkner, ok | 13:24 |
jelkner | lol | 13:24 |
jelkner | did you see my correspondence with ingrid? | 13:25 |
jelkner | we are ready to vote on aea-pac | 13:25 |
replaceafill | jelkner, yes | 13:25 |
mr_german | jelkner, well, I'll start from now to work on that | 13:26 |
jelkner | sweet | 13:26 |
mr_german | replaceafill, if I have some question you'll able to help? | 13:26 |
mr_german | to help me* | 13:26 |
replaceafill | jelkner, is that going to be a "do you approve these bylaws"? yes/no | 13:26 |
replaceafill | jelkner, type of thing? | 13:26 |
replaceafill | mr_german, i can, but you can reach out to ORMD directly | 13:27 |
replaceafill | mr_german, since jelkner already introduced you | 13:28 |
mr_german | replaceafill, mm ok | 13:28 |
jelkner | replaceafill, +1 | 13:28 |
jelkner | on both bylaws and mr_german reaching out directly ;-) | 13:28 |
replaceafill | :) | 13:28 |
replaceafill | jelkner, i think i can have the AEA instance set up on Friday | 13:29 |
replaceafill | jelkner, for voting | 13:29 |
*** mjsir911 has joined #novawebdev | 13:29 | |
replaceafill | jelkner, i'm testing the helios email integration now with ORA | 13:29 |
mr_german | jelkner, replaceafill I forgot to tell you that aea php version is using "http://aeava.net/" | 13:30 |
mr_german | not the tendenci version | 13:30 |
replaceafill | mr_german, ah thanks | 13:30 |
mr_german | ;) | 13:30 |
replaceafill | mr_german, we need to redirect that | 13:30 |
dsmall | replaceafill, "pip install djangorestframework==3.6.4" worked | 13:31 |
replaceafill | dsmall, cool | 13:32 |
dsmall | replaceafill, thanks for the help | 13:32 |
replaceafill | dsmall, sorry for all the confusion generated :) | 13:32 |
dsmall | replaceafill, np | 13:32 |
replaceafill | dsmall, that's usual with dependencies | 13:32 |
jelkner | replaceafill, $$ sent | 13:34 |
replaceafill | jelkner, transfer confirmed, thank you very much! | 13:35 |
jelkner | Apparently the maximum amount allowed is $2999 | 13:35 |
jelkner | how is lelkneralfaro dealing with that? | 13:35 |
jelkner | and how long does one need to wait before doing it again? | 13:36 |
replaceafill | jelkner, it's a daily maximum iirc | 13:36 |
jelkner | oh, np than | 13:36 |
jelkner | mr_german, please hang on for another day | 13:36 |
replaceafill | jelkner, we paid mr_german hours for may already | 13:37 |
jelkner | s/than/then ;-) | 13:37 |
mr_german | jelkner, to start? | 13:37 |
mr_german | oh | 13:37 |
jelkner | no! | 13:37 |
jelkner | to get paid ;-) | 13:37 |
mr_german | I already got paid jelkner | 13:37 |
jelkner | good, mr_german, if you're happy, i'm happy! ;-) | 13:40 |
dsmall | replaceafill, Check it out so far: http://i.imgur.com/cTXIKuy.png | 14:06 |
replaceafill | dsmall, ah nice | 14:07 |
replaceafill | dsmall, so what's your plan? are you sticking to that package? | 14:07 |
dsmall | replaceafill, not sure how to nest comments yet, but I am working on it | 14:07 |
replaceafill | dsmall, or are you going to test the others? | 14:07 |
dsmall | replaceafill, I think this one will work well | 14:07 |
replaceafill | dsmall, cool | 14:07 |
dsmall | replaceafill, I wasn't able to get one of the other ones to work, and I haven't touched the third | 14:08 |
replaceafill | dsmall, please test if it has moderation options | 14:08 |
replaceafill | dsmall, ah ok | 14:08 |
dsmall | replaceafill, this one has a lot of moderation options | 14:08 |
dsmall | replaceafill, you can moderate out works and spam | 14:08 |
replaceafill | dsmall, cool | 14:08 |
dsmall | replaceafill, words** | 14:08 |
replaceafill | dsmall, can the comments be approved first? | 14:09 |
replaceafill | dsmall, i mean, like not publishing them initially | 14:09 |
replaceafill | dsmall, until an admin allows it | 14:10 |
dsmall | replaceafill, im not sure, I can look into it for you | 14:10 |
replaceafill | dsmall, please | 14:10 |
replaceafill | dsmall, thanks! good progress | 14:10 |
replaceafill | dsmall, hope you're still enjoying it :) | 14:10 |
dsmall | replaceafill, I am :), thanks for all of the help | 14:10 |
replaceafill | dsmall, np | 14:10 |
dsmall | replaceafill, check out this doc about moderation: http://django-contrib-comments.readthedocs.io/en/latest/moderation.html | 14:14 |
replaceafill | dsmall, ah ok | 14:20 |
replaceafill | dsmall, but that's for extending the system, right? | 14:20 |
dsmall | replaceafill, ya, I believe so | 14:20 |
replaceafill | dsmall, initially we only need to know what the system can do out of the box | 14:20 |
replaceafill | dsmall, we can talk about it once you're more familiar with it | 14:21 |
dsmall | replaceafill, alright that works | 14:21 |
dsmall | replaceafill, I will probably mess around with it a bunch tomorrow/tonight | 14:22 |
replaceafill | dsmall, +1 | 14:22 |
*** mjsir911 has joined #novawebdev | 14:35 | |
replaceafill | ACTION goes to get lunch, bb in ~40 | 14:53 |
jelkner | ACTION signs off for the day | 15:01 |
*** mjsir911 has joined #novawebdev | 15:12 | |
*** mjsir911 has joined #novawebdev | 15:59 | |
*** mjsir911 has joined #novawebdev | 18:42 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!