Accessed Date and URL are treated as null for articles with a page range unless you check the "Include URL" box in the cite tab of the Zotero preferences. With that box checked, you should get the access date (which, frankly, is a bit of a silly requirement for DOIs, which are meant to be stable).
You also want to make sure to change the style ID, otherwise your style will get overwritten by the standard APA style regularly https://www.zotero.org/support/dev/citation_styles/style_editing_step-by-step#change_the_style_title_and_id.


Edit: actually, as written, you're explicitly not including the access date when there is a URL -- as the original APA style does, you're first testing for the DOI, if that's present print it with the https://doi.org/ prefix, and only when you don't have a DOI you're printing. You'll want something like:
<macro name="access">
<choose>
<if variable="DOI" match="any">
<text variable="DOI" prefix="https://doi.org/"/>
</if>
<else-if variable="URL">
<text variable="URL"/>

</else-if>

</choose>
<group delimiter=" " prefix=" (" suffix=")">
<text term="retrieved" suffix=":"/>
<date variable="accessed" form="numeric"/>
</group>
</macro>

Do you have the include URL box checked?

can you link to your current version of the style? This should work