Migrate admin/config/services/twitter/settings page form to D8

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

juampynr’s picture

Version: 7.x-6.x-dev » 8.x-1.x-dev

Moving to 8.x

plopesc’s picture

Status: Active » Needs review
FileSize
2.93 KB

This patch creates the settings page following the new Routing system.

This patch does not work for now because variables should be converted con config system.

This patch is a first approach. Form is shown, but does not work.

Regards

juampynr’s picture

Status: Needs review » Needs work

Wrong patch.

plopesc’s picture

Right patch

juampynr’s picture

Missing hook_menu() changes for this to work.

plopesc’s picture

hook_menu() included

juampynr’s picture

Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

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

  • Commit c0c0dfe on 8.x-1.x by xurizaemon:
    Issue #2011876. Update settings form for recent Drupal 8 changes.
    

  • Commit 18ff081 on 8.x-1.x by xurizaemon:
    Issue #2011876. Update settings form for recent Drupal 8 changes.
    
  • Commit d1bf4a1 on 8.x-1.x by xurizaemon:
    Issue #2011876. Depend on OAuth (not oauth_common).
    
    In #2012008 OAuth...