It would be great if shariff integration is also available for Drupal 8.

Todo for a stable release:
- #2663614: Drupal 8 Cache API integration (!)
- #2663610: Idea: Make settings configurable per block
- #2698447: Add composer support

CommentFileSizeAuthor
#4 release_a_drupal_8-2426839-4.patch59.99 KBdigitaldonkey
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

criz’s picture

Assigned: Unassigned » criz

Working on this.

digitaldonkey’s picture

I'm working on this too since yesterday. Would be great to have a D8 Branch.
@criz : Maybe we can share our progress somehow.

jcnventura’s picture

Issue tags: +SprintWeekend2016
digitaldonkey’s picture

Status: Active » Needs work
FileSize
59.99 KB

So far for SprintWeekend2016:

  • Block support is working (but not fully tested)
  • Config UI is working.
  • Drush support is not implemented yet
  • Field support is not implemented yet
  • Some more todos in Code, some dead code requiring cleanup.
  • upgrade-info.html for cleanup Information

Would be great id the maintainer can open a D8 Branch.

criz’s picture

Thx, will have a look on it, merge with my work so far (if useful) and open a D8 Branch then...

  • criz authored aa1dc7c on
    Issue #2426839 by digitaldonkey, criz: Release a Drupal 8 version
    
criz’s picture

criz’s picture

Issue summary: View changes
criz’s picture

Title: Release a Drupal 8 version » [meta] Release a Drupal 8 version
Version: 7.x-1.0 » 8.x-1.x-dev
Grienauer’s picture

After enabling shariff on a new d8 site it gives me following error when visiting the module settings page.

Recoverable fatal error: Argument 2 passed to shariff_help() must be an instance of RouteMatchInterface, instance of Drupal\Core\Routing\CurrentRouteMatch given in shariff_help() (line 15 of sites/default/modules/shariff/shariff.module).

  • criz authored 1f9d992 on 8.x-1.x
    Issue #2426839: Fix missing RouteMatchInterface, leftover call of submit...
criz’s picture

Okay thx, should be fixed now.

Grienauer’s picture

Tested and works now!
Would be great to have an info if the library is not installed on the module config page.

criz’s picture

Issue summary: View changes
Status: Needs work » Needs review

First alpha release is in the queue.
I consider it already rather stable. However, it can need some reviews for a beta release.

Drush and field support can be added later as features.

criz’s picture

Status: Needs review » Fixed
criz’s picture

Added field support. When no mature bug reports are coming in the next days we should publish a full module version.

Drush support is not on my priority list, but can be contributed at any time.

Status: Fixed » Closed (fixed)

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