- Issue created by @Steven Snedker
- Assigned to dev2.addweb
- 🇮🇳India dev2.addweb
Hi,
The Media Remote page ( https://www.drupal.org/project/media_remote → ) will be updated by the maintainer only. So, it is handled by the maintainer. Now, please focus on the README.md file.
- Issue was unassigned.
- Status changed to Needs review
9 months ago 7:22am 2 July 2024 - 🇮🇳India dev2.addweb
Hi, I have updated README.md and created MR. Please review it.
- 🇩🇰Denmark Steven Snedker
Great. Though it looks to me as if the numbered list has only 1s and no 2,3,4...
- 🇮🇳India dev2.addweb
Hi @steven-snedker,
I wrote in README.md file 1s for all, because it's automatically convert to sequence number list by README.md.
For more information regarding README.md file structure and format please visit standard template → - 🇮🇳India riddhi.addweb
I have tried and tested the Read Me file
I have compared the Read Me file with the Doc, attaching the doc link for the same.
Doc Link:- https://www.drupal.org/docs/develop/managing-a-drupalorg-theme-module-or... →Everything seemed alright when we compared it in the README.md file, attaching the URL for the same. And moving it to RTBC.
README.md File URL:- https://git.drupalcode.org/issue/media_remote-3456494/-/blob/3456494-imp... - Status changed to RTBC
8 months ago 7:50am 3 July 2024 - Status changed to Needs work
5 months ago 10:01am 21 October 2024 - 🇳🇱Netherlands ekes
This patch is specific to configuring for Buzzsprout. Which is actually really handy for me, as that's what I was doing :-) But it should probably be an example addition to the readme, rather than a replacement for the generic description that there is. I'd guess more specific examples might be good, I've not tried if there are other specific differences.
- 🇩🇰Denmark Steven Snedker
The generic description is very brief and does mention Buzzsprout in passing:
Configuration & Use
Install this module as usual.
Create a new Media Type, select "Remote Media URL" as Media source
Important: Configure the display, selecting as formatter in the URL field your desired provider. For example, if you are creating a "Buzzsprout" media type, select "Remote Media - Buzzsprout" as formatter for the URL field in the "Manage display" tab.My proposed resolution is guaranteed to show newbies the ropes by
a) providing them with an actual working example (that's Drupal Module Gold)
b) taking them through all 7 steps in the correct sequence.The smart ones can probably write something other than Buzzsprout on their first run. The less smart ones can get a working example and change bits and pieces until it works with another provider. Or make a new one in the vein of Buzzsprout.
A generic Configuration & Use where Buzzsprout is changed to [PROVIDERNAME] and [PROVIDERURL] is less readable. And won't necessarily work.
I also like the bit in step 6:
Click "Add" (clicking "Save" will make stuff explode). Click "Save". Click "Insert selected". (Scroll to the bottom and) click "Save".
It's probably still correct (but I haven't tested since June, so who knows).
My conclusion: this patch is useful for everyone and a great improvement.