Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Oct 04 2022 22:13
    argordmel commented #219
  • Oct 04 2022 10:12

    kilbot on master

    update language packs (compare)

  • Oct 04 2022 09:23

    kilbot on master

    Update woocommerce-pos-admin-de… (compare)

  • Sep 29 2022 14:25
    runestone74 closed #254
  • Sep 29 2022 14:25
    runestone74 commented #254
  • Sep 29 2022 14:24
    runestone74 opened #254
  • Aug 02 2022 12:29
    ikbel opened #253
  • Jan 27 2022 09:16
    Vellimack commented #25
  • Dec 17 2021 10:36
    protechcode opened #252
  • Nov 04 2021 13:05
    kilbot synchronize #214
  • Nov 04 2021 13:05

    kilbot on 0.4

    0.4.26 remove old docker, use wp-env i… (compare)

  • Oct 17 2021 14:38
    kilbot closed #251
  • Oct 17 2021 14:38
    kilbot commented #251
  • Sep 29 2021 00:47
    zidingz opened #251
  • Sep 24 2021 07:53
    m-zat commented #244
  • Sep 24 2021 07:38
    rleegwater commented #244
  • Sep 15 2021 06:07
    m-zat commented #244
  • Sep 09 2021 10:19
    m-zat commented #244
  • Sep 09 2021 08:51
    m-zat commented #244
  • Aug 07 2021 08:53
    kilbot commented #250
Paul Kilmurray
@kilbot
most components are services that is
again .. I think some of this is housekeeping
and it’s great to have someone to explain this too because it makes realise that I can refactor and simplify this a lot
Thomas Mery
@thomasmery
happy to help ;)
Paul Kilmurray
@kilbot
while we’re here, I might as well finish the lib folder .. because we have covered most of it :)
Thomas Mery
@thomasmery
ok - listening
Paul Kilmurray
@kilbot
the lib folder is for shared code
so a Sub App will likely use some shared components and behaviours
Thomas Mery
@thomasmery
behaviors are a Mn thing right?
Paul Kilmurray
@kilbot
behaviours are a Marionette thing, they just let you abstract common View methods
yep (beat me too it)
Thomas Mery
@thomasmery
http://marionettejs.com/docs/v2.4.7/marionette.behavior.html
This message was deleted
Paul Kilmurray
@kilbot
most of it is the same
the Views have been simplified in Mn 3.0
Thomas Mery
@thomasmery
changed the link
Paul Kilmurray
@kilbot
I will update to Mn 3.0 … I just have to get the next version of WooPOS out first :)
before I use any standard Backbone or Marionetter object, I extend it first so we have our own POS specific copy
these go in the config folder
this just gives us a chance to customise things as necessary
Thomas Mery
@thomasmery
understood
Paul Kilmurray
@kilbot
so you will notice even the Behavior object is extended
even though I don’t change anything
Thomas Mery
@thomasmery
saw that
Paul Kilmurray
@kilbot
but if at some stage we decide every POS behaviour should have a particular method, we only have to change one file
so the app.Route we extended yesterday is actually the config Route which just extends a plain Mn Object
as with the app.Router and the app.ItemView
Thomas Mery
@thomasmery
looking at them
Paul Kilmurray
@kilbot
there is some bad housekeeping in that folder also
receipt-view should be a component
it is just a view, but it is used in several sub apps
Thomas Mery
@thomasmery
do you want to start filing issues for these kind of things so maybe we can keep track of them and I could try to cleanup refactor - or it would be better not to waste time on this for now
Paul Kilmurray
@kilbot
I will leave it for the moment … a lot of these choices are arbitary. I think it would be good to discuss these things again once you are more familiar with the code
Thomas Mery
@thomasmery
ok
Paul Kilmurray
@kilbot
you might have some suggestions on simpler organisation
I have a lot of legacy ideas :)
Thomas Mery
@thomasmery
as you should :)
Paul Kilmurray
@kilbot
ok, so that covers the lib folder .. there is also some utilities in there, which is pretty self-explanatory
Thomas Mery
@thomasmery
I'll go through this on the week end
Paul Kilmurray
@kilbot
that’s it for the boring stuff, lets clone your example from yesterday and work with some data
don’t worry too much about absorbing all this on first look
Thomas Mery
@thomasmery
no, just familiarizing myself
Paul Kilmurray
@kilbot
I think you’ve got all the experience necessary to work on the project, I just want to give you a look around the project, gives you and I a chance to talk and then we can discuss hourly rate and hours if you want to get involved :)
Thomas Mery
@thomasmery
yep
I'm quite interested as you can see, I hope
Paul Kilmurray
@kilbot
yeah, it’s great to meet a WordPress developer with some javascript chops!!
so you should be able to command line to .html/wp-content/themes/twentysixteen/
Thomas Mery
@thomasmery
I'm there
getting back the changes from yesterday
Paul Kilmurray
@kilbot
then git clone https://github.com/thomasmery/wcpos-tutorial.git