Comments

  1. This looks soo cool thankyou – I’ll try to do this next batch of bills with it and see how I get on. (of course it means I actually have to keep track of and report my hours properly myself rather than just telling every one else to do it!)

  2. 1. Please see the attached agenda that needs to be added to the pg for this years conference.

    2. Please see the link (akomkeselecture2024.ticketleap.com/akom-kese-2024/) that is captured in the attached picture. We need a button that will have this link embedded so that individuals can buy tickets to the conference.

    3. Finally, please replace the phone no. for the Council of Elders (on the CoE contact pg) with (929) 515-6001‬.

    Akom Kese 2024 Agenda Website Version

  3. @Rob, I have added the functionality for Hours Liason to Live site now. You can test it.
    Here are the things to consider.
    1) you will need to add Liaison hours to tasks which are created by enabling is_liaison checkbox https://prnt.sc/4yA4JMVifqcb
    2) The client payments for liaison hours need to be recorded separately in backend by enabling is_liaison checkbox again https://prnt.sc/P95xKnLCe3XB

    There is a separate section for liaison hours on hourly log page now. Your hourly balance will show there.

  4. Well it works really well thankyou:-) For the layout please can we have:
    Width of captcha code field and display both much less – reduce they fit 4 characters comfortably
    Mobile – put ‘Email Address’ field below ‘First Name’ field. Leave captcha as it is now
    Desktop – leave email/first name fields as they are but put captcha code field/display horizontally beside eachother.
    Please set mobile/desktop breakpoint to wherever the desktop view can no longer fit everything in one line.

    How clear is that? Sorry I’m rubbish at drawing visuals…

    1. Hi Rob,

      The css requirement is done.

      Please check for mobile and desktop versions.

      Thanks,
      Shubham

      1. Looks really good thankyou – OK I have one last change… Please add a css ‘hide’ by default for both catcha and captcha field (send button should collapse left). Then when someone tyes a character into the ‘First Name’ field, they should change to display. That way it will look nicer until someone types in it:-)

  5. Hi Rob,

    Please can you adapt the above to work on product 13720 and write the price as A$47
    –The task is done for the product id that you gave but the id is 13702 and it is only work for one product and the price change from “EURO” to “AUD”. check the below screenshot.
    https://prnt.sc/-oZzw-KEWvN2

    Let me know if you need anything on this.

    Thanks,
    Shubhamoy

    1. Ok thanks, I dont understand why a new web page is needed to only add an email.
      Maybe there is more information for the page which we didn’t get.

      1. >> The council of elders contact form should be a contact form with fields. The information should go to the provided +email address. The link should be added as a pulldown option under the coun6of of elders.

        So basically they want a dup of the contact page (contact form and all) but going to a different email called Contact the Council of Elders. With link as submenu under council of elders menu.

        Basically it’s a sub organisation with separate contact deets!

        I said you’d do it this week – hope that was realistic.

        I’m about to send Shubham a ton of work on several projects by the way:)

  6. prepare the Save the Date (Akom Kese) – I am not sure what it means and where it needs to show. Can you provide more details, along with some screenshots?

    1. I’m guessing these won’t be massively time consuming so please let me know if you’re not going to be able to have a go on them before mid next week so I can set his expectation:-). I’ll get back to you on a ton of other things over the next couple days too hopefully…

      1. Ok I remember this task from last year and the year before that. I suppose they conduct these events annually.
        It will require couple of small changes on different pages and menu.
        Do you have staging for this project? You can then assign this to Shubham.

        Otherwise I can make these changes directly on live by Wednesday or Thursday.

        1. Annoyingly no staging sorry. So we’ll go for the weds/thurs option. He also says >> For the Council of pg, please create a pg with fillable fields and forward inquiries to the email address that I have given you. (I can see it in one of the screenhots in this thread so we have got it…)

          Have a look at the wp/plugin versions while you’re in there – if its really old the maybe I can get him to pay for an hour to set up a staging and test some upgrades:) (then it’d be easier to ask Shubham to do things too…)

          Thanks!

  7. >> Would you prepare the Save the Date (Akom Kese) to reflect the following dates 02Aug2024 to 04Aug2024?

    Hopefully makes sense? I know they’re small updates sorry.. I’ll check if he needs to do another pre-pay. Might be worth showing Shubham round this site for half an hour so he can take over…

  8. all prices gone on live site for dvds… do we just have to blame this one on resources/hosting – they’re all still there on the dev server which is on my hosting!

    1. If you check on individual DVD page https://www.strokesolutions.co.uk/product/successful-stroke-survivor-dvd-volume-1/ then the prices are still showing. This means the patch I had added worked partially.

      Do we have any response from woocommerce support about this transient issue? If we know why this happens only on live server then there might be a simple solution like using a hook.

      Also to prepare a patch I have to test the plugin code directly on live which is risky.

      1. I didn’t get time to post a woo support request…. Given it happens on live and not staging I think I wann just blame it on resources and use that as an impotus for Tom to get onto a proper server – shared hosting is not good enough for this site! Understood re patch/live risk. Thankyou!

  9. I checked & uploads nicely now.

    re. missing code. that may have been my fault though I’m not sure. sorry if so – I’m managing several sets of changes at the same time. Karan and I are talking about how to make it easier. Meanwhile there is another change client wants. Please can you add a show/hide condition to the contract status indicator you added. It needs to show alway *except* when the instructor viewing is enabled for SLT. There is some condition like is_slt_enabled() – you might need to do a search for it…

    Lastly – there’s a problem now on staging that when I view a patient profile it just shows up a blank section (screenshot attached). On live you get the appointments tab. Please don’t use any extra JS to fix this. It should just work naturally so one of the changes relating to the patient template must have broken it as all othersare deployed.

    1. Hi Rob,
      for this requirement: Please can you add a show/hide condition to the contract status indicator you added.

      i added this in dev server: https://www.awesomescreenshot.com/video/25191381?key=5fb37bfbd547a60112be6ccfdaf87516

      if i misunderstood, please correct me.

      And for the second one:
      it is done on stage server. video for your reference: https://www.awesomescreenshot.com/video/25191783?key=72b3e6aba034a036c21996dea4c92d59

      As per your request no additional js added, the issue was caused by another two js files, “e.indexof” error. fixed those.

      And one more thing Please provide me the admin, instructor and patient dummy details for live site. i had those but now it is changed.

      1. Hi Shubham, Apologies for taking so long to get back to you. Partly I had to travel but mainly we had some security problems that meant I had to do a lot of work on several sites – that’s why the stroke sol live password changed. Do you have an email address I could add you to our slack channel? that’s where Karan and I have been swapping things like passwords.

        Thankyou re sorting out the e.indexof error. I was worried that one might be really hard to figure out… How long did it take? Please can you update your hours on this task?

        Contract status: no sorry… Please have a look at this sentence: It needs to show always *except* when the instructor viewing is enabled for SLT. There is some condition like is_slt_enabled() – you might need to do a search for it…

        There is a type of instructor that is ‘SLT enabled’ – actually right now only one called Olivia Hogdkinson. She has a totally different set of contracts to sign for her patients so the ‘contract status’ indicator is nonsense for her. Therefore we’d like to hide it for her. There will be more SLT instructors like her. The condition is something like is_slt_enabled() I think. If you have trouble finding it ask me and I’ll look.

        Thanks! (I also wanted to mention, we have a fair bit of work on several projects to come… it’s me slowing everything down so I’m trying to re-organise so the flow of work is more steady for all of us!)

        1. Hi Rob,

          1. Contract status: no sorry… Please have a look at this sentence: It needs to show always *except* when the instructor viewing is enabled for SLT. There is some condition like is_slt_enabled() – you might need to do a search for it…
          —–I did search for this function “is_slt_enabled()” and find some different function names like this: https://prnt.sc/a8MdeOmV4OXA and did this change: https://prnt.sc/QYnM3WwncsaH. If i got it wrong the please elaborate the task with screenshots.

          2. Thankyou re sorting out the e.indexof error. I was worried that one might be really hard to figure out… How long did it take? Please can you update your hours on this task?
          —– It didn’t take much time, that’s why i didn’t update the hours.

          And also i got the password from the slack, thank you for letting me in slack.

          Thanks,
          Shubham

          1. Not exactly but we’re nearly there I think.
            Case 1: this is the default. It displays that contract status column and displays the traffic lights for every patient
            Case 2: $slt_enabled = intval( get_user_meta( $this->get_id(), ‘instructor_slt’, true ) ); (from class.ss-user.php) If this condition is true for the instructor, the please simply hide the column.

            I thought there was a function that would return the result of this condition but it seems we never needed one outside the class. So I found the line above. Maybe you need to add another funotion to theclass? Does this make more sense?

            Thanks!

        1. Hi Rob,

          I saw your response just now, sorry for the late reply.

          1. Do you have an email address I could add you to our slack channel?-
          –It’s shubhamoybandhuroy@gmail.com , You can add me on slack on this email.

          2. Contract status: no sorry… Please have a look at this sentence: It needs to show always *except* when the instructor viewing is enabled for SLT. There is some condition like is_slt_enabled() – you might need to do a search for it…
          — Apologies for the misunderstanding, I will look into it tomorrow and will update you.

          3. I also wanted to mention, we have a fair bit of work on several projects to come… it’s me slowing everything down so I’m trying to re-organise so the flow of work is more steady for all of us!
          — I am eagerly waiting for new tasks as well and want to contribute.

          Thanks,
          Shubham

          1. Hi Shubham, please could youtake a look at the contract status indicator again. Does this explanation make sense:

            Case 1: this is the default. It displays that contract status column and displays the traffic lights for every patient
            Case 2: $slt_enabled = intval( get_user_meta( $this->get_id(), ‘instructor_slt’, true ) ); (from class.ss-user.php) If this condition is true for the instructor, then please simply hide the column.

            I thought there was a function that would return the result of this condition but it seems we never needed one outside the class. So I found the line above. Maybe you need to add another function to the class? Does this make more sense?

  10. Please note also on dev site password for robtest oser if that’s the one your using has changed to –user_pass=’8N3K9nYkw’
    On live it is –user_pass=”8N3=*K9nYk=w”

  11. Hi, also a related issue- this instructor is enabled for telesessions and yet the column in her patient list is not showing ‘online’ as a possibility for sessions. I think there’s something wrong with the criteria for showing that column. Instructor name is fleur howard. Patient is Graham betteridge

  12. Not sure if you get notifications for new tasks so commenting here:-) Also for point 2 I’d like each payment request record to be able to request both types of hours (orif one requests zero then the hours counter ignores that client payment record)

      1. Hi – when I upload to the applications section it behaves as expected. But when I try to upload a jpg to the contracts section it says ‘disallowed’ as it would for a pdf.

      2. hiya – just re-posting in case you missed it: Hi – when I upload to the applications section it behaves as expected. But when I try to upload a jpg to the contracts section it says ‘disallowed’ as it would for a pdf.

          1. Video looks great thanks! What do you mean by ‘also my code was override from dev server’? Did I mess it up by eiting something ese maybe?

  13. The issue with prices not showing seems to be a new one.
    It could be that the variation stock data caching was need cleared in database.
    I tried updating the woocommerce to latest version on my local but couldn’t replicate the issue.

    I have updated the plugin on staging site and updated a single product page to clear caching. This seems to have fixed the issue.
    We can wait for few days, if the issue pops up again then we will have more details what might be causing it.

    1. I reckon we could just put this live now with caveats – pls can you either do it or let me know which files to save me blasting the whole woocommerce dir:)

  14. Sorry for delay, I have updated the image now.
    These tasks are very minor. It would be good if we can have 2-3 tasks combined which I can assign to Shubham.

  15. I had a hard time finding your latest comment and I thought maybe it is no longer an issue :p

    Through the email notifications I could find your comment.

      1. Hiya – I can see all products are available which is great – but I’m not seeing the prices on the dvds page: https://dev.strokesolutions.co.uk/product-category/dvds-and-anytime-viewing/ as compared to the live version – i think this is what happens when they go unavailable so probably we need a fix for where it shows the prices too.

        Same goes for price display on individual product page.

        Good thing is I think this proves to me that it works (ie you’re making ‘unavailable’ products available!)

  16. Still couldn’t see the remote branch… But deployed on staging/merged/committed/tested/deployed on live. Feature is totally awesome – I think he’s going to like it. I think next priority for us has to be to get into a routine with git and give Shubham etc a clear procedure. Still at least we’re not fighting with tasks/time records/bills so much anymore:-)

  17. I have completed this tasks and pushed changes to git repo in “CPD” branch.

    Please see final screenshots for instructor dbs upload fields and frontend carousel view:
    File Upload: https://prnt.sc/nGbTj_KFpD7G
    Frontend: https://prnt.sc/J3TZ83zoRSby

    Please test and let me know if there are any fixes needed.

    Also one thing to note that this uses new database logic to store file data so previous instructor files will not be included.
    Instructors will need to upload the dbs files again.
    If Tom wants old files to be migrated to new logic then I will have create a separate import script for that. It will be better to create that as separate task.

    1. Hi – I can’t find any new branches in the remote repo – Do you mean the old git repo vlado/ivan used to use? Since the connection to swift from bitbucket broke I’d not seen any changes in it from you and assumed it was not being used. Since then I’ve been committing them to my own repo (ssh details in strokesolutions/.git/config). I no longer have access to Vlado’s bitbucket. I don’t know why but I can’t see the workspace anymore. Please can we use the new repo so all the work Shubham and I have been doing the last couple of months counts – for now maybe just push the changes as files and I’ll commit them if you prefer.

      Secondly looking at these screenshots although it looks like you got the feature spot on – I think maybe we’re a lot of messages along from the original task description and something’s been forgotten along the way. Namely that the DBS feature is to be left exactly as-is except that he wants icons on the front end (for the existing three files). The CPD section is currently just a text box and is to be replaced with the feature that it looks to me like you’ve now coded for the DBS section.

      So no migration would be necessary – maybe just keep the text field visible in the back end so Tom can see the old data?

      Hope that makes sense!

      1. I dont know why I completely misunderstood the task.
        Anyway, there weren’t lot of changes to make.. I only had had the change the prefixes used in code and reposition some fields.

        I have updated the code for CPD section and added an update for thumbnail view of DBS on frontend.

        For repo, can you provide me ssh link? Is it using a service like git or bitbucket?

        For the meantime, I have attached files with code changes

        StrokeSolutions

        1. hiya – no probs – it’s up to me also to make sure you’re clear so you can get on with the development… I knowit was a complicated one.

          I’ll test within next 24hrs – thanks so much.

          [core]
          repositoryformatversion = 0
          filemode = true
          bare = false
          logallrefupdates = true
          [remote "origin"]
          url = ssh://zh2948s2@zh2948s2.rsync.net/data2/home/zh2948/zh2948s4/stroke_solutions.git
          fetch = +refs/heads/*:refs/remotes/origin/*
          [branch "master"]
          remote = origin
          merge = refs/heads/master

          above is the strokesolutions/.git/config – my backup server hosts git repos:) I run my entire life via git repos/diary/machine configs etc Actually feels kinda strange to use it for code!

          For authentication just use same private key you use for all my server stuff.

          Hopefully it interacts ok with what you’re running – I’ve only ever used it in cli as you can imagine… Let meknow!

          1. That’s cool!

            I have pushed the changes to cpd branch in you git repo.
            Hopefully it will easier for you to check changes now.

  18. Hi Rob,
    This task is done from my end.
    Please check it and let me know the status of the task.
    Thank you.

        1. looked great to me thankyou – client has yet to give me feedback. hopefully will have more work for you next week – just need the time myself to write it up:-)

  19. Nice one with the screenshot;) Hm so I think you’re saying most efficient time/cost implementation of infinite file uploads is sub-optimal db representation – however if user removes item 4 it can be empty and therefore we simply don’t show empty items on front end – can we do same on back-end? ie once item 4 is deleted just not display it on either back or front end? Or are we stuck with an empty slot on back-end that doesn’t display on front-end until we fill it? If that’s a major time/money saver then maybe that’s ok. If we don’t have to display deleted slots anymore then maybe we just don’t number them – so no-one would ever know slot 4 was deleted?

    I will ask Tom if he wants to give each cpd a text name to show underneath. They can be arbitrarily named unlike dbs which are always the same…

    Re icons – i checked on the live site and could find pdfs, jpgs and pngs but no docs. How about a default icon for pdf/doc/docx and then thumbnail for png/jpeg? Yeah I guess we change the text to ‘DBS pt 1’ etc beneath each icon?

    1. Ok I am working on alternative solution now based on your suggestion. I will have to assign a different meta value for storing these file Ids so they can be added and removed in an array set.
      This may take some additional time as it will be using new logic.
      Please see attached new screenshot of design which you can show to Tom.

      Also, can you tell me if these dbs data is used anywhere else on site apart from instructor profile? I might have to change changes for other functions also.

      1. Thanks I think this is much better – understood re time… Also, each one will need a text field and a date picker ‘Description’ and ‘Date’ – to display on front end under icon eg. 13/10/2023: ARNI ASSESSMENT or 07/04/2023: ARNI/UCL MEETING – he’ll also wants a carousel on the front end – nothing animated or anything but to display the first 3 or 4 (whatever fits best) then an arrow to load next 3 or 4 – then a back arrow – maybe give me a revised total estimate is a good idea. I’m sure he’ll do it whatever but I’m thinking it’s more than initial guesstimate was…

        Can it be done by christmas? 😉 Hopefully not too much else on the board for you if mpyke are a bit quieter – I’ve got ANKpodcast and Shubham/me can do other SS tasks… Then hopefully come January we’ll be in a position to get him talking about template sites again…

        1. I am working on these changes now.
          The new title and date fields will also require updating database rows for each file and changing the instructor layout design.
          It will take approx 11-12 hours total for this task now.

  20. I investigated the code for this and basically it is possible to create infinite fields for uploading files https://prnt.sc/yM39BpU9Efcj
    However the way data is stored in backend it is wp_usermeta with specific position id to store the uploaded file id https://prnt.sc/hwR4bCJVQr8T

    I can add a remove file button to empty the file field and set metadata empty however it will not be possible to shift position of other fields.
    For example if the instructor adds files for 4,5, and 6 fields in backend, then decide to remove 4 then we can only set it as empty and not move 5 to 4 and 6 to 5
    On the frontend however I can still show proper order of files.

    Secondly for thumbnails, can you tell me the different format of files which are uploaded. Are they mostly doc files or can there be pdf and jpg/png images? Any other format?
    i might have to include separate icons for doc/pdf files.
    Also do you want to keep following text for uploaded files on frontend ? https://prnt.sc/SAJ2OUHvH0tt

  21. Hiya – please can you put a check on each row to see if it has the wordpress option ‘make this post sticky’ checked – and if it does stick that nice pinned icon next to it? So basically if we check this option then wordpress automatically puts it at the top of the feed – but the modification I’m asking for would add your icon to it. Maybe put it on the left so it looks obvious… THanks!

  22. @Ash I have fixed the Billing Clinic Address error on checkout field. The form is submitting now. Please test and let me know if there are any other issues.

    @rob I have taken backup of live so you can deploy this ticket and booking form changes to live once they are approved because these tasks required updates on many files.
    After that if there are any css fixes needed then I will manually add them.

      1. Hi Rob. These event changes are ready to go live. Can you also please put the booking form changes for this task live at the same time (not sure how to tag you in it): https://freedomonlineservices.com/task/1974/

        When you do it can you also please copy this page over to the live site because it’s linked to on the updated booking form: https://mpyke.freedomonlineservices.net/srh-training/terms/

        Might want to do a cheque order or something afterwards to check it’s working. Thanks!

        1. ok – will try to fit this in monday afternoon – if not then tuesday afternoon. looks to me like deploying the code will deploy both tasks so i’ll just copy over the page and give the shop a quick 1pound donation:)

          I’m set as a moderator on these tasks so i get notifications same as you. We’re not sophisicated enough for tagging around here:-)

          1. Thanks Rob. It’s just occurred to me that I asked Karan to make some changes on the live site (the ones numbered 1-6 in comments below). Will those get overwritten if you copy the code from the dev site?

            Sorry if that makes things more complicated but need to keep those changes and they haven’t been done on the dev site.

          2. haha… it happens… I got to this earlier today than expected so had just finished deploying/checking when I read that. I think I’ve fixed it though pls double check for me. Basically I’ve deployed all code from live site and re-applied single_event.php, single_product.php and assets/css/style.css to both live and staging. It looked to me from the modification dates like they were the ones Karan modified on live site. I’ve also imported the terms page and checked a purchase. I think we’re good now… Ash please give it the once over though as you’re more familiar with these changes.

            Karan, there’s a .git folder now on live and staging. Again, like ss dev site there’s a .git/config that defines the remote. I don’t know how you interact with git (cli like me? hopefully…:-) but I’m thinking a good workflow for direct live css would be (on live) git add . ; git commit -m ‘something’ ; git push origin master . Then on staging git stash ; git pull origin master ; git stash apply Your thoughts?

          3. Hi Rob, I won’t pretend I understand much of what you said but it seems some things didn’t get transferred to the live site.

            1. Start Time and End Time fields in events aren’t there on live, it’s still the old single ‘Time’ field instead.
            2. Booking form on live isn’t the same as the new dev site version. For example the ‘Professional Registration Number’ field is required on live but not required on dev, and field labels for ‘Participant Information’ section haven’t been updated on live (they still say delegate instead of participant).
            3. ‘Booking terms and conditions’ link on live booking form goes to page on dev site. Need it to open https://margaretpyke.org/srh-training/terms/

            Also the ‘Clinic Address’ field has disappeared on the dev site booking form. I’m sure it was there when I checked it the other day so not sure what’s happened there.

            The changes I mentioned that Karan made to the live site are still there though so that worked fine. Also the product updates worked because the Start Time and End Time fields are there and the shortcodes for them worked in the booking confirmation email I received when making an order.

            UPDATE: Just noticed another problem, don’t know if it’s related to today’s updates. ‘On demand’ events aren’t showing events of the same type in the ‘Other options available’ section at the bottom of the event pages, such as https://margaretpyke.org/event/lecture-on-demand-fertility-and-conception/ – it’s showing events of type ‘Webinar’ instead. I checked the settings at https://margaretpyke.org/wp-admin/admin.php?page=acf-options and it looks like it’s set up correctly so I don’t know why it isn’t working correctly? It’s working as it should for event types ‘SRH’ and ‘Webinar’.

          4. Karan I think we need your help here – I can see the code for the Event field group includes ‘start time’ and ‘end time’ on live same as on staging. But I can also see as Ash says if you go to the dashboard we only have ‘time’. Do we need to do a dashboard import/export on the acf groups? (seems a bit weird though as they are there in the code…) I’m guessing probably any other issues are related to this – I checksummed the code – I really did deploy it right this time (unlike last week on Tom’s site;)

    1. Thanks Karan, the [start_time] and [end_time] shortcodes are working fine in the automatic emails. Found a couple of issues with the booking form so I’ll report those on the other task.

      1. Hey Rob/ Ash… sorry i didn’t take into consideration booking task during deployment. There were couple of updates needed on dahboard.
        I have added them now. Also updated the style on both sites.

        Can you please test now and let me know if you have any other issues?

        1. Thanks Karan. Booking form looks fine now. Please can you help with the following:
          1. Start Time and End Time event fields are in a different format on live than they are on staging. Please can you change the fields on the live site to behave the same as the fields on the staging site? On staging they are text fields which then display in event boxes in the format ‘9.30am – 4.30pm’.
          2. Please can you change the ‘Booking terms and conditions’ link on the live booking form to https://margaretpyke.org/srh-training/terms/ – it’s currently linking to the page on the staging site.
          3. On the live site, ‘On demand’ events aren’t showing events of the same type in the ‘Other options available’ section at the bottom of the event pages, such as https://margaretpyke.org/event/lecture-on-demand-fertility-and-conception/ – it’s showing events of type ‘Webinar’ instead. I checked the settings at https://margaretpyke.org/wp-admin/admin.php?page=acf-options and it looks like it’s set up correctly so I don’t know why it isn’t working correctly? It’s working as it should for event types ‘SRH’ and ‘Webinar’.

        2. thanks Karan, we’ll get better at this tag-team thing… I’m guessing you didn’t touch the git repo? I supposed I can just commit the live, stash the staging untracked files and pull repo to staging at this point right?

          1. Hi Rob, yes I didn’t touch the git repo. You can update the changes for live on staging now.

    1. Thanks Karan. Found two issues:
      1. It’s possible to successfully make an order without ticking the ‘I agree to the booking terms and conditions’ box. Please can you set it so this must be ticked before an order can be placed, with a validation message ‘You must agree to the booking terms and conditions’ and highlight the field on the form if it’s not ticked when the form is submitted.
      2. I selected the ‘Other please specify’ option for the ‘How did you hear about the course?’ field, but the text I entered in the box didn’t show on the order. Instead it says ‘Other please specify’.
      Also a couple of minor changes while you’re doing this:
      3. Please change the linked text to include the word ‘booking’ as well, so the link is ‘booking terms and conditions’ instead of just ‘terms and conditions’.
      4. If possible please set the ‘booking terms and conditions’ link to open in a new tab.

      Thanks!

        1. Hi Karan, the client has come back with one field label change for the booking form:

          1. Please change ‘Yes! Sign me up to receive emails about future training courses. (optional)’ to ‘I would like to receive emails about future training courses’

          Can do it on live site but maybe it’s better to change it on staging as well at the same time so we don’t have different versions on each site? Whatever you think is best. Thanks!

          1. I have updated this now on both staging and live:
            1. Please change ‘Yes! Sign me up to receive emails about future training courses. (optional)’ to ‘I would like to receive emails about future training courses’

      1. I made a couple of tiny changes then put it live – thankyou, great job! I’ll get on to answering your question on the other task by the weekend ifI can… Also, an apology – the problem with the ‘/users’ page breaking was nothing to do with your export code. Client deleted a bunch of users and I exported them to file ‘users.sql’ and it broke the theme router!

  23. Hi Shubham – checked and deployed! However I had to do some fixes. The theme including your changes on the other ‘export’ task is now in stroke-solutions-bu folder in themes folder. Something was breaking the /users page so I moved the news tab changes to theme backup. In the process I: 1) moved tab-news.php to partials/shared/ 2) fixed a aria-controls= descriptor

    I noticed that now all the default tabs work correctly, so thankfully we don’t need the change to the footer.php. On other projects we cut corners/add quick fix hacks a lot (me especially…)but no this one we cannot afford to as it is so complicated.

    Anyway I’ll mark this one ‘done’ and we’ll fcus on the next task…

    1. Hi Rob,
      Hope you’re doing well.
      Is there any fault in my code?
      Please let me know it so that i can start improving it.
      And also thank you for your info.

      1. Hi don’t worry about it – this site is extremely complicated. Between us we’re replacing Karan while he works on something else and it was never going to be easy! He is extremely skilled and has been working on this site for a year… Anyway, in this case – the default loading tabs needed to work without you adding a js fix in the footer:-) I think maybe one of the css attributes was wrong – I corrected a couple – ‘aria-controls’ and also I think I added a ‘fade’ attrbute… or maybe it was one of the other code changes you made for the other ‘exports’ task that broke it. I’ll try to get back to you on your questino in the ther task this week and we’ll have another go!

  24. Hi Ash,

    Sorry for delay, my laptop had broken so I needed to send for repair.

    I have added the start_time and end_time fields for products which will then show in course emails when using these shortcodes [start_time], [end_time]
    I have also updated event time fields.
    Please try testing and let me now if you find any issues.

    @Rob just for info most of the updates have been done in code so you might need to take backup when syncing staging with live so that changes are not overwritten.

    1. hiya -so…. if I want to put these latest changes live are you saying I need to merge them with some changes you’ve also made on live? I’m thinking to get git working on this project soon… That said, maybe you can do live deployment for now? 🙂

    2. Thanks Karan.

      Time fields for events working fine.

      I can set times for products but I can’t test the [start_time] and [end_time] shortcodes in emails at the moment because I can’t submit an order. When I fill out the booking form and try to submit it I get the error ‘Billing Clinic Address is a required field.’ even though I’ve entered text in the Clinic Address field. Looks like you’ve done some of the requested changes to the form already so you’re probably still working on it, but please let me know when it’s working again though so I can check the time shortcodes. Thanks!

  25. Hi Rob,
    I have made the recent changes on dev server, Please check.
    And also about your query
    if I were to deploy just the template file changes you made to live site, would it work? In other words, did you make any changes in the wp dashboard that would also need to be duplicated?
    Yes just upload the template files for the frontend and also add this file to the particular folder in live site(This folder is basically for the backend file to show the news data): https://prnt.sc/-DhYXlqtBp2q
    And i didn’t change anything in the dashboard.

    1. Instructor profiles now correctly show a default tab – but patients and subscribers still do not. Please can you fix this. Patient should show ‘appointments’ and subscribers should show ‘videos’.

      Thanks for the info re php files!

      1. Hi Shubham – do you know when you’ll get a chance to look at this? > Instructor profiles now correctly show a default tab – but patients and subscribers still do not. Please can you fix this. Patient should show ‘appointments’ and subscribers should show ‘videos’.

  26. Hi Shubham, by ‘pinned post’ client meant the facility of wordpress where checking ‘make this sticky’ in screenshot attached the post goes to top of list. This seems to work fine with your news page so please remove your pinned function. Please keep a copy of thecode thoughas I havean idea for it later on another feature…

    1. Hi Rob,
      Thanks for the clarification, So should i change the existing code to show the sticky posts at the top of the posts list?
      And for the pinned posts functionality i have given the option to user who can pinned the posts according their needs(and it is differentiated by their Id’s), I think i misunderstood that.

      Please tell me if i need to do the sticky part or just remove the pinned post code for now?

      1. It already shows sticky posts at the top of the post list I just tried it. Yes please remove your pinned post functionality but keep a copy of the code as we may be able to use it somewhere else.

          1. Hi Shubham,

            Almost there:-) When I view https://www.strokesolutions.co.uk/profile/david-wren/ the default tab ‘appointments’ appears but when I view https://dev.strokesolutions.co.uk/profile/david-wren/ on staging site it no longer loads a default tab. Please check also default tab for instructors/subscriber role users.

            Also – if I were to deploy just the template file changes you made to live site, would it work? In other words, did you make any changes in the wp dashboard that would also need to be duplicated?

            Thanks!

  27. Hi rob,
    I have some queries for the above task.
    1. In the present functionality there are two amount field in the excel sheet named “onl:amount and in:amount”;
    –So should i sum the two fields and show the total amount in a extra field or something else?
    2. Same thing for KICKBACK field.
    3. and for the rest, i have done some R&D on the functionality, It is slightly working but need to differentiate b/w appliaction page1,page2 ,page3, same for the rest.

    the overall task will take 4-5 hours to complete.

    Please approve the time and answer the queries.

    1. Hiya – apologies it took me so long answering this.
      1. ‘onl’ means online, ‘in’ means in person (ie instructors doing lessons over zoom vs doing lessons face). So yes please sum them. 1. is onl:amount + in:amount and 2. is onl: ss kickback + in: ss kickback
      2.yep
      3. great.

      Yeah time/money is what it is. Client just got a huge grant in september so he’s keen to get things done and not too bothered re cost. He’ll be a lot less generous when it’s running out next summer! That said, if you and I replacing Karan means higher costs initally I may have to discount him somewhat. We’re doing ok so far!

      1. Hi Rob,
        Thank you for clarify my queries, I will start the work from tomorrow.
        and I am bit confuse about this point of yours:- He’ll be a lot less generous when it’s running out next summer! That said, if you and I replacing Karan means higher costs initially I may have to discount him somewhat. We’re doing ok so far!

        1. thankyou -sorry re my unclear statement. I have to ‘chat’ when I talk to clients so it is a habit… But having tried to learn a second language myself I should keep my task messages simple (After many years living in Italy I still struggle to work in the language!).

          All I meant was we are not tight on budget right now and the time estimate you provided is totally fine – client is focussed on getting tasks done right now.

          But that will probably change in a few months time when he has spent a lot of his money! It happens every year… He’s an amazing guy but not the greatest at managing budgets…

    2. also please remember your modified theme files for this task are in wp-content/themes/strokesolutions-bu because I was debugging… Please feel free to modify the theme again now. Lastly, also please keep your hours well up to date wednesday/thursday/friday as I have to send clients invoices. Thanks!

  28. Thanks Karan. 1-6 look fine to me so I’ll let them know.

    For 7 and 8, they aren’t using timings on product programmes anymore so we need to pull the data for [start_time] and [end_time] shortcodes from somewhere else. We already need that information to show on the event boxes so I was hoping it could be taken from the event fields when they’re set up. Course types are set per product and then products are assigned to an event so don’t know how complicated that would be. If it’s a problem then the [start_time] and [end_time] could be set for products instead but it would need new fields for them (best next to the date field at the top), as well as the new fields for events because the times still need to show in the event boxes as well.

    1. Ok in that case, we will need to put separate time fields for products because autoresponder emails use course types and product order information.

      To make things easier, 1 update i can do is if event time fields are added and primary product is set then the primary product can automatically take event time data for its fields after first save.

      Let me know if this seems like a possible solution.

      1. It’ll be fine to just add the time fields to both products and events so I can set them separately. If the time fields for products can be next to or below the Course Date field at the top then it’ll be easy to remember to set them. They should be unrequired fields because not all courses have start and end times. Thanks.

  29. Regarding these 2 tasks:
    7. Please change ‘Time’ field for events to ‘Start Time’ and add another ‘End Time’ field. The event boxes for event types ‘SRH’ and ‘Webinar’ should then display the information from those two fields in the format ‘9.30am – 4.30pm’.
    8. After the above is done please update the [start_time] and [end_time] shortcodes for the automatic emails (which are set up under ‘Course Types’) to insert the times from those fields.

    I checked the code and currently start_time and end_time shortcodes for automatic emails are generated from itinerary section in products https://prnt.sc/tGgycEYI12NR
    Do you instead want to generate it based on event template start time and end time?
    Events are associated with products based on conference type custom field. Can you confirm this also? https://prnt.sc/XXfVBy0Y58p0

    1. >Events associated with products
      Hmm no I don’t think so. I think each ‘Event’ has a ‘Primary Product’. (we don’t care about the secondary/tertiary anymore – deprecated!) The field you screenshotted/highlighted shows how events are related to ‘Course Type’. So if some-one goes to an event page they buy a product but then the course-type determines all the email responders…

      Hoping Ash has a handle on the rest;-)

  30. The blog posts have been assigned blog or podcast category in backend. The category archive template was missing so I have added it now.

    I have received the paypal payment from frank

  31. Hi @Ash, good to see your comment on FOS!

    @rob I will take on Mpyke tasks for now since I wrote the code for this site and I am aware of the different conditional logic.

    I have updated these changes on live, please check:
    1. For ‘On Demand’ events, such as https://margaretpyke.org/event/margaret-pyke-webinar-18-july-2023/ please can you make the heading lines for date and speaker slightly smaller and less bold. Maybe try h4 and see if it looks ok?
    2. For all event boxes please can you make the titles (e.g. ‘Emergency Contraception Encapsulated’ here https://margaretpyke.org/srh-training/lectures-on-demand/) slightly smaller, so they are a bit smaller than the ‘Lectures on Demand’ heading at the top of the page.
    3. For ‘On Demand’ event pages, please change ‘Recorded On:’ to ‘Recorded on:’
    4. For ‘On Demand’ event pages, please change ‘Topics Include:’ to ‘Topics include:’
    5. For ‘SRH’ event pages, please change ‘Topics Covered:’ to ‘Topics covered:’
    6. For ‘Webinar’ event pages, please change ‘Topics Covered:’ to ‘Topics covered:’