CommentFileSizeAuthor
#6 socialfeed_info_port.png10.95 KBdhruveshdtripathi
#2 porting_info_file_to_D8-2657982-2.patch772 bytesznak

Comments

Hemangi Gokhale created an issue. See original summary.

znak’s picture

znak commented
Assigned: hemangi.gokhale » Unassigned
Status: Active » Needs review
StatusFileSize
new porting_info_file_to_D8-2657982-2.patch772 bytes

Patch for this issue

Aanal.addweb’s picture

Aanal.addweb commented
Status: Needs review » Needs work

@Znak, Thanks for the patch, it works well as per the requirement for porting to d8, but after applying patch, it should remove the old info file as your provided patch.
Let me know your views on it.

znak’s picture

znak commented
Status: Needs work » Needs review

@dhwani.addweb, hello. I removed *.info file, because in Drupal8 *.info.yml file replaces *.info file.

Aanal.addweb’s picture

Aanal.addweb commented

@Znak,Yes you provide patch for the same but after applying it didn't remove the *.info file.

dhruveshdtripathi’s picture

dhruveshdtripathi commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new socialfeed_info_port.png10.95 KB

It removes .info file and replaced with .info.yml file.

hemangi.gokhale’s picture

hemangi.gokhale
she/her/hers
Primary language English
Location Portland, 🇺🇸
commented
Status: Reviewed & tested by the community » Closed (duplicate)

Thanks for your efforts guys, but there's no need to port single files as there's already an issue aiming to port the whole module into D8, hence marking this as Duplicate