Problem/Motivation

Currently, the module page only contains information for the Drupal 7 release. We should add further installation / usage information on the module page for the 8.x-2.x release.

Proposed resolution

Add further installation / usage information on the module page for the 8.x-2.x release.

Also the other supported / recommended versions should be described and by their porpose, plan & differences.

Example for the module page:

Versions

  • 3.x: Active development, Drupal 9/10 support, Drupal 8 support may end soon. Added field items wrapper & further functionality
  • 8.x-2.x: Security fixes only, Drupal 8 support, no Drupal 10 support.
  • 7.x-2.x: Active development, but maintenance fixes only. 7.x-1.x with further fixes. See comparison here.
  • 7.x-1.x: Original Drupal 7 release, security fixes only.

Issue fork fences-3303084

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

Grevil created an issue. See original summary.

anybody’s picture

Title: Add installation / usage informations for Version 8.x-2.x to the module documentation / page. » Add proper documentation for 8.x-2.x on the module page & README

Indeed, the module page seems to target 7.x and is confusing for Drupal 8, which may be more relevant due to EOL.

anybody’s picture

Title: Add proper documentation for 8.x-2.x on the module page & README » Add proper documentation for all versions on the module page & README.md
Issue summary: View changes
anybody’s picture

Issue summary: View changes
anybody’s picture

Issue summary: View changes

Added the template for the module page to copy & paste HTML (edit) after review.

anybody’s picture

Issue summary: View changes
grevil’s picture

Looks good! I would add "Drupal 9/10 support" for 3.x. Everything else looks fine

anybody’s picture

Issue summary: View changes
Status: Active » Fixed

Thanks, I'll copy that over.

anybody’s picture

As I can see, you already did :) Thanks!

grevil’s picture

Status: Fixed » Active

Also, I removed "Added field items wrapper & further functionality", because future functionalities can be seen in the current update change. Otherwise, we always have to add newly added functionalities to the module page.

Now only the readme needs to be reworked.

grevil’s picture

Status: Active » Needs review

I created a simple README.md, @Anybody should we include more detailed information on how to use this module? Or is the current implementation enough?

anybody’s picture

Version: 8.x-2.x-dev » 3.x-dev
Assigned: Unassigned » grevil

@Grevil better than nothing already! :D

I think you should create a best practice structure, perhaps there's a template at Drupal.org or you can have a look at very well maintained and much used modules.

Then fill in the current / additional content which helps users starting with fences from scratch.

Afterwards, I think it would be best if @thomas.frobieter makes final changes from a typical Drupal Designer's perspective.

Let's only do this for 3.x so far... For 7.x I'm still unsure about the difference between 1.x and 2.x and which one to use for the future ;)

grevil’s picture

Component: Code » Code (fences)
Assigned: grevil » Unassigned
Status: Needs review » Needs work
anybody’s picture

Title: Add proper documentation for all versions on the module page & README.md » Update module page and README.md to be up to date

Only 3.x is available now. All other versions have been deprecated. So we should update the module page and README.md to document the current state and fine!

lrwebks made their first commit to this issue’s fork.

lrwebks’s picture

Status: Needs work » Needs review
anybody’s picture

Status: Needs review » Needs work

@lrwebks please finally add a paragraph about fences presets, like on the module page.

Then ensure the module description has everything important and we're done!

lrwebks’s picture

Status: Needs work » Needs review

I'm really not sure why, but after hitting “Update Fork”, for some reason GitLab treats those merged commits as separate commits made on this branch, which clutters things up a bit in the MR…

Regardless, I have added the section about fences presets and made sure that the introduction covers all the necessary info that was present in @grevil's README, just compressed down a bit to the essential info.

Also, PHPUnit failure is unrelated.

anybody’s picture

Status: Needs review » Reviewed & tested by the community

@lrwebks the wrong branch had been selected. Needs to be 3.x but 8.x-2.x was selected.

anybody’s picture

@lrwebks did you update the module page?

lrwebks’s picture

Re #20: I see! That's probably why Drupal Core is about to introduce a main branch instead of many version.x branches (I personally get the reason for version branches, but I still find them to be very confusing sometimes)

Re #21: I did not and cannot because I'm not a maintainer.

anybody’s picture

@lrwebks sorry I made you maintainer and already updated the module page.

  • anybody committed d9d5afa1 on 3.x authored by grevil
    Issue #3303084: Add proper documentation for all versions on the module...
anybody’s picture

Status: Reviewed & tested by the community » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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