Updated README.txt. Please find below the attached patch.

CommentFileSizeAuthor
#17 interdiff_15-17.txt494 bytesdeepak goyal
#17 3150756-17.patch1.68 KBdeepak goyal
#15 interdiff_14-15.txt1.26 KBmeena.bisht
#15 3150756-15.patch1.66 KBmeena.bisht
#14 interdiff_12-14.txt1.25 KBmeena.bisht
#14 3150756-14.patch1.66 KBmeena.bisht
#12 interdiff_5-11.txt609 bytesdeepak goyal
#12 3150756-11.patch1.56 KBdeepak goyal
#5 3150756-5.patch1.58 KBmeena.bisht

Comments

rahulrasgon created an issue. See original summary.

rahulrasgon’s picture

rahulrasgon commented
Title: Allow HTML to render through iframe. » Update README.txt
Component: Code » Documentation
meena.bisht’s picture

meena.bisht commented
Assigned: Unassigned » meena.bisht
Status: Active » Needs work
meena.bisht’s picture

meena.bisht commented
meena.bisht’s picture

meena.bisht commented
Issue summary: View changes
StatusFileSize
new 3150756-5.patch1.58 KB
meena.bisht’s picture

meena.bisht commented
Assigned: meena.bisht » Unassigned
meena.bisht’s picture

meena.bisht commented
Status: Needs work » Needs review
dipakmdhrm’s picture

dipakmdhrm commented
Status: Needs review » Reviewed & tested by the community

Looks good enough.

rahulrasgon’s picture

rahulrasgon commented
Status: Reviewed & tested by the community » Needs work 
TODO
----

- add more extension like jpg , jpeg , png , pdf etc .
- integrate iframes to render the file data which is loaded in this field .

@meena.bisht Can you update the extensions correctly. Extensions should be rar, tar.gz etc

Thanks

deepak goyal’s picture

deepak goyal commented
Assigned: Unassigned » deepak goyal
deepak goyal’s picture

deepak goyal commented
Assigned: deepak goyal » Unassigned
Status: Needs work » Needs review

Hi @rahulrasgon
Updated patch please review.

deepak goyal’s picture

deepak goyal commented
StatusFileSize
new 3150756-11.patch1.56 KB
new interdiff_5-11.txt609 bytes
meena.bisht’s picture

meena.bisht commented
Assigned: Unassigned » meena.bisht
Status: Needs review » Needs work

Some minor changes need to be done :
-add more extension like jpg, jpeg, png, pdf, etc.
- integrate iframes to render the file data which is loaded in this field.

The first letter should be capital and adding some extra information to it.

meena.bisht’s picture

meena.bisht commented
Assigned: meena.bisht » Unassigned
Status: Needs work » Needs review
StatusFileSize
new 3150756-14.patch1.66 KB
new interdiff_12-14.txt1.25 KB

Please find the patch attached with all the updates.

meena.bisht’s picture

meena.bisht commented
Assigned: Unassigned » meena.bisht
StatusFileSize
new 3150756-15.patch1.66 KB
new interdiff_14-15.txt1.26 KB

Removed extra spacing

Lal_’s picture

Lal_
He\Him
commented
Status: Needs review » Needs work 
+++ b/README.txt
@@ -1 +1,68 @@
+The module has no menu settings. For this add new field in any of the content-type,

Seems confusing could you rephrase

deepak goyal’s picture

deepak goyal commented
Assigned: meena.bisht » Unassigned
Status: Needs work » Needs review
StatusFileSize
new 3150756-17.patch1.68 KB
new interdiff_15-17.txt494 bytes

Hi @Lal_
Updated patch please review.

Lal_’s picture

Lal_
He\Him
commented
Status: Needs review » Reviewed & tested by the community

Much better moving to RTBC

rahulrasgon’s picture

rahulrasgon commented
Status: Reviewed & tested by the community » Fixed

Thank you everyone for working on this issue.

Patch #17 committed and pushed to 1.0.x branch.

Thanks

rahulrasgon’s picture

rahulrasgon commented

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.