Codeinwp / Codeinwp/optimole-wp
Allow downloading DAM attachments to user's website
Nobody has claimed this yet.
- Dominant language
- PHP
- Stars
- 72
- Forks
- 14
- Avg merge
- 2d 9h
- Merged PRs (30d)
- 15
Description
What problem does this address?
Right now, we treat DAM attachments as we do with offloaded images (they are available to download to the user's website on the media page when switching to the list layout).
We should allow users to download them to the website and use them as plain attachments, how we do for the offloaded images.
What is your proposed solution?
- Allow processing these attachments through the rollback process and pull them to the user's website;
- Remove the DAM import flag we use right now after rollback: https://github.com/Codeinwp/optimole-wp/blob/master/inc/dam.php#L148
- Make sure we replace the URLs in the website content;
Will this feature require documentation? (Optional)
None
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reading the rollback processing and inc/dam.php around line 148, then trace how DAM attachments are handled on the media page in list layout. The work is done when rollback pulls DAM attachments to the user's website, removes the DAM import flag, and replaces their URLs in website content. No tests or additional entry points are named.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php
- Domain
- backend
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100