Hi, this is a comment.
To get started with moderating, editing, and deleting comments, please visit the Comments screen in the dashboard.
Commenter avatars come from Gravatar.
Hi, this is a comment.
To get started with moderating, editing, and deleting comments, please visit the Comments screen in the dashboard.
Commenter avatars come from Gravatar.
actually two other questions that occur to me – have the pages (blog posts in particular) actually been migrated properly? And have the comments been migrated?
hiya – how’s this looking? Doable? (fingers crossed – gotta be better than rhythm passport i’m hoping;) – I realise you can’t give me any serious time estimate but if you had a wild guess it’d be helpful… I can mention it when I invoice him for 10hrs or whatever…
Hiya, now frank’s paid I’m putting my mind to his work again;-)
Re these seo links – i’d like http://franklove.net/something to go to frank-love.com/something v maybe that means redirecting the 404page in this case.
And also, the main menu pages need to redirect to their counterparts – ie franklove.net/category/blog needs to go to frank-love.net/blog (you can see the main menu links by going to frank-love.com/something
Lastly, I’m wondering if you could go in there with a free security plugin and make it as hard to hack as possible. I don’t think he’s going to want to pay to upgrade it ever – but it’ll probly have to be there at least a couple of years.
1. Contract just whirrs for regular telessesions user
Steps to reproduce:
a) create new instructor user
b) activate him for telesessions
c) log in as instructor and got to contracts section. Then click to sign.
Result:
– blank page except for close box and whirring thing in middle
Questions:
– has this impacted live site also as changes were made in docusign?
– do other stroke solutions contracts still work? I could not sign the regular stroke solutions contract for my instructor either
2. Instructor Link exists to sign contract that does not change upon activating SLT
Steps to reproduce:
a) activate an instructor for SLT
b) log in as that instructor and go to the ‘telesessions’ tab
c) click on the option ‘here’ to go to the telesessions welcome page
d) click to sign up for telesessions
Result:
– blank page as above – I presume because this link does not change
Notes:
– I believe this link is generated by a shortcode if that helps…
3. Multiple warnings on Intructor dashboard
Steps to reproduce:
a) log in as instructor
Result:
– warning: attempt to read [roperty ID on null in…
Questions:
– is this only because we’re in debug mode (ie won’t show on live)?
– does it matter? 🙂
4. Multiple warnings on Patient Dashboard
a) create new patient and don’t assign to anyone yet – just log in
Result:
– warning undefined array key ‘in person’ in…
Questions:
– nothing to do with us this one right? 🙂
– do you think it matters or even shows on live without debug mode?
5. Patient Link exists to sign contract that does not change upon activating SLT
a) Create patient and connect to instructor signed up as SLT
b) log in as patient and got to telesessions tab
c) click to say you’re ready to sign up for telesessions (and go to the patient telesessions welcome page)
d) click to sign contract for telesessions
Result:
– blank/close box whirring screen
6. No ‘Contract’ tab between orders and transactions when you log in as patient
Steps to reproduce:
a) log in as rob@littleguybigheart.net,testpassword
Notes:
– have I missed something – I can see it’s there in your screenshot… ?
Also, the ‘Instructor Assigned’ email needs disabling for Slt patients… I missed that one… Maybe best we get the rest fixed and live first as its not critical. Looking at the above again I’m wondering if a lot of it is ‘dev site’ related issues. But we have to know for sure…
that’s awesome thanks. Tom’s agreed to try to get contracts/new email details for next week for us. So I’ll oheck monday morn. email notification for this comment arrived this time:)
Hiya, I tested again and all looks fixed thankyou. Please ping me asap for when you can put it live (in a morning before I get up if possible so I can do a few checks). Only small thing I saw was when you’ve signed a contract as an instructor, the instructor telesessions welcome page says ‘contract signed thankyou’ instead of giving you a link to sign one. Whereas when you’ve signed as a patient (for slt at least – i haven’t tested upper limb) it just continues to offer you the possibility to sign. Probably not worth delaying going live but will need fixing at some point I think. I’ll attach a screenshot of logged in patient.
OK Forget my bug reportand screenshot! It was me gettin, confused. It’s all good to go!!
ok in that case pls could you make the change to that dvd/streaming bundle (removing streaming access) and also check through the rest of the dvds/bundles to make sure they’re not giving people free streaming access also:-) We only found out about this one because he was honest – can’t imagine people telling us if they don’t have to…
please can you check all the physical dvd products to make sure none of them give out logins in this way. It looks like a lot of them are set like this…
Can you send me a copy of contract? has he uploaded it somewhere?
If he has simply replaced his oldcopy in docusign then I dont think it will work and instructors will notice new contract. We will need to update the related contract keyword in code files for this.
Hiya, looks like dashboard still only lets you set class dates for men (screenshot attached) and also looking at the ‘processing’ email it doesn’t look like the text from 6 is included. Did you hard code it? Would love to get 4 and 5 to him this week then send him instructions/bill if possible…
Hey Rob, I am able to understand the problem discussed in the task but I dont know what solution you want me to work on. Do you want me to research on paytia and integrate this on site and does it need to work with stripe? This seems like a complex research based task which may take some time.
I think at this point ignore paytia – that was just background info – but please could you look into the code to see where that checkout page is for adding sessions – both for admin and for the instructor it is on strokesolutions.co.uk/plans and have a think about what would be necessary to implement saved cards so people could just click to pay next time without entering card details. It’ll need some stripe research too. Tom’s out of hospital and at home feeling a lot better. Once he’s back working and cleared his inbox a bit I’ll send him another invoice!
Hi, this is a comment.
To get started with moderating, editing, and deleting comments, please visit the Comments screen in the dashboard.
Commenter avatars come from Gravatar.
Hi, this is a comment.
To get started with moderating, editing, and deleting comments, please visit the Comments screen in the dashboard.
Commenter avatars come from Gravatar.
screenshot of pages to redirect
pages_to_redirect
actually two other questions that occur to me – have the pages (blog posts in particular) actually been migrated properly? And have the comments been migrated?
Here’s the toggle location on the instructor profile
A_-_Toggle_Location_-_Instructor_Profile
hiya – how’s this looking? Doable? (fingers crossed – gotta be better than rhythm passport i’m hoping;) – I realise you can’t give me any serious time estimate but if you had a wild guess it’d be helpful… I can mention it when I invoice him for 10hrs or whatever…
Hiya, now frank’s paid I’m putting my mind to his work again;-)
Re these seo links – i’d like http://franklove.net/something to go to frank-love.com/something v maybe that means redirecting the 404page in this case.
And also, the main menu pages need to redirect to their counterparts – ie franklove.net/category/blog needs to go to frank-love.net/blog (you can see the main menu links by going to frank-love.com/something
Lastly, I’m wondering if you could go in there with a free security plugin and make it as hard to hack as possible. I don’t think he’s going to want to pay to upgrade it ever – but it’ll probly have to be there at least a couple of years.
to see what he means
pastedImage
I have updated this now
Screenshot_17
great thankyou – he is happy
adding a comm”nt cos i think that’ll make it email you abôt this task:-)
attached here – ignore the wp errors at the top from acf – have to talk about that separately
frank-casinos
This should be working fine now. I found that it had letsencrypt SSL enabled under domain settings which I have enforced now.
If you go to site it should take you to https – akancenter.org/
brilliant thankyou!
Issues Found:
1. Contract just whirrs for regular telessesions user
Steps to reproduce:
a) create new instructor user
b) activate him for telesessions
c) log in as instructor and got to contracts section. Then click to sign.
Result:
– blank page except for close box and whirring thing in middle
Questions:
– has this impacted live site also as changes were made in docusign?
– do other stroke solutions contracts still work? I could not sign the regular stroke solutions contract for my instructor either
2. Instructor Link exists to sign contract that does not change upon activating SLT
Steps to reproduce:
a) activate an instructor for SLT
b) log in as that instructor and go to the ‘telesessions’ tab
c) click on the option ‘here’ to go to the telesessions welcome page
d) click to sign up for telesessions
Result:
– blank page as above – I presume because this link does not change
Notes:
– I believe this link is generated by a shortcode if that helps…
3. Multiple warnings on Intructor dashboard
Steps to reproduce:
a) log in as instructor
Result:
– warning: attempt to read [roperty ID on null in…
Questions:
– is this only because we’re in debug mode (ie won’t show on live)?
– does it matter? 🙂
4. Multiple warnings on Patient Dashboard
a) create new patient and don’t assign to anyone yet – just log in
Result:
– warning undefined array key ‘in person’ in…
Questions:
– nothing to do with us this one right? 🙂
– do you think it matters or even shows on live without debug mode?
5. Patient Link exists to sign contract that does not change upon activating SLT
a) Create patient and connect to instructor signed up as SLT
b) log in as patient and got to telesessions tab
c) click to say you’re ready to sign up for telesessions (and go to the patient telesessions welcome page)
d) click to sign contract for telesessions
Result:
– blank/close box whirring screen
6. No ‘Contract’ tab between orders and transactions when you log in as patient
Steps to reproduce:
a) log in as rob@littleguybigheart.net,testpassword
Notes:
– have I missed something – I can see it’s there in your screenshot… ?
Also, the ‘Instructor Assigned’ email needs disabling for Slt patients… I missed that one… Maybe best we get the rest fixed and live first as its not critical. Looking at the above again I’m wondering if a lot of it is ‘dev site’ related issues. But we have to know for sure…
I will need to clean this data in database but I am not able to access the cpanel login. It seems password has been changed
Screenshot_20
yeah i’ve asked him for it twice – glad its not just me… we havean excuse to delay him now:)
Dev site is bit different from live site because wp_Debug is set to true and lot of upload documents are not available.
3 and 4 are related to wp_debug
1 is occuring because a new instructor has to be approved by admin before any contract signing
I will check the rest and get back to you
I have added fixed for 1 and 6
for 1 since it was using new demo account of docusign on dev site so all template documents were not uploaded which I had to take from live.
6 was an actual bug for SLT in patients profile and has been fixed
I have pushed fixes for 2 and 5 also. Please check
that’s awesome thanks. Tom’s agreed to try to get contracts/new email details for next week for us. So I’ll oheck monday morn. email notification for this comment arrived this time:)
Hiya, I tested again and all looks fixed thankyou. Please ping me asap for when you can put it live (in a morning before I get up if possible so I can do a few checks). Only small thing I saw was when you’ve signed a contract as an instructor, the instructor telesessions welcome page says ‘contract signed thankyou’ instead of giving you a link to sign one. Whereas when you’ve signed as a patient (for slt at least – i haven’t tested upper limb) it just continues to offer you the possibility to sign. Probably not worth delaying going live but will need fixing at some point I think. I’ll attach a screenshot of logged in patient.
OK Forget my bug reportand screenshot! It was me gettin, confused. It’s all good to go!!
Screenshot_20220820-100420
Yes I think it could be because of streaming addon set in DVD variation
Screenshot_31
ok in that case pls could you make the change to that dvd/streaming bundle (removing streaming access) and also check through the rest of the dvds/bundles to make sure they’re not giving people free streaming access also:-) We only found out about this one because he was honest – can’t imagine people telling us if they don’t have to…
please can you check all the physical dvd products to make sure none of them give out logins in this way. It looks like a lot of them are set like this…
I have added updates for 1,2,3 and 6 on live. please check
Screenshot_37
users can only checkout after login
Screenshot_35
this is great thankyou – i’ll check on the site tomorrow. how are we doing for hours on this?
Can you send me a copy of contract? has he uploaded it somewhere?
If he has simply replaced his oldcopy in docusign then I dont think it will work and instructors will notice new contract. We will need to update the related contract keyword in code files for this.
ok I read it and thought about it… Makes no sense to me either! I’ll ask him…
I have updated this, removed the posts now. Please check on your end.
great thankyou – please can you also remove these 5? then i think he’s finally clean…
malware_scan_export
Hiya, looks like dashboard still only lets you set class dates for men (screenshot attached) and also looking at the ‘processing’ email it doesn’t look like the text from 6 is included. Did you hard code it? Would love to get 4 and 5 to him this week then send him instructions/bill if possible…
Screenshot_20220824-092337
and this
Screenshot_20220824-092032
@Rob basically we need to add a postcode field and zoom button to zoom the map to the postcode area. Is that correct?
It should be doable but I will need to check the google api code for this
ok please spend a bit of time in the api docs and give us a rough time estimate.
@Rob I have made the updates for 4, 5 and 6
https://prnt.sc/VAWDbsrUCVpu
https://prnt.sc/vCS2JG9hiOxF
a
Screenshot_20220904-121251
another
Screenshot_20220904-120950
@Rob I have created the api fields and deployed this on staging. you can test it here https://dev.strokesolutions.co.uk/users/?user_role=map
It seems to be working so I think geocode is already enabled for the api
If you still want to change the api you can do it here
https://dev.strokesolutions.co.uk/wp-admin/admin.php?page=acf-options-settings
Screenshot_34
I have fixed the podcast page and created the search page layout. You can check here
https://frank-love.com/?s=history
Hey Rob, I am able to understand the problem discussed in the task but I dont know what solution you want me to work on. Do you want me to research on paytia and integrate this on site and does it need to work with stripe? This seems like a complex research based task which may take some time.
I think at this point ignore paytia – that was just background info – but please could you look into the code to see where that checkout page is for adding sessions – both for admin and for the instructor it is on strokesolutions.co.uk/plans and have a think about what would be necessary to implement saved cards so people could just click to pay next time without entering card details. It’ll need some stripe research too. Tom’s out of hospital and at home feeling a lot better. Once he’s back working and cleared his inbox a bit I’ll send him another invoice!
screenshot
neurogripper_payment
screenshot
Event_vs_transaction
another
Event_vs_trasaction_2