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

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
StatusFileSize
new2.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

StatusFileSize
new4.17 KB

Right patch

juampynr’s picture

Missing hook_menu() changes for this to work.

plopesc’s picture

StatusFileSize
new4.73 KB

hook_menu() included

juampynr’s picture

Status:Needs work» Fixed

Committed. Thanks!

https://drupal.org/user/282415

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...