Part of marketplace improvements.
In order to introduce Featured section of Marketplace we need to:
1. add field to organization node type with the value Featured
2. copy service providers display, add filter on Featured value
3. modify basic view to not show Featured organizations
Work in progress: http://tvn-drupal.redesign.devdrupal.org/marketplace
To-dos:
Fix urls in top Categories navigationThere is some number in () on the left and since I've added new argument to the view - urls of Categories need to be adjusted:
- column Services has correct urls "drupal-services/[featured|all]/[Service]"
- column Sectors has urls of the type "drupal-services/all/[featured|all]/[Sector]" - it should be "drupal-services/[featured|all]/all/[Sector]"
- column Countries served has urls of the type "drupal-services/all/all/[featured|all]/[Country]" - it should be "drupal-services/[featured|all]/all/all/[Country]"Add active trail to sidebar menu items on sub pages featured/* and all/*Remove (comment out) code for DA membership check and auto-adding to the listing.
Checkbox “Request listing in marketplace” should only automatically create an issue in the Webmasters queue.
Status of the new issue should be "needs review" instead of "active".Build a view of all org nodes listed in the marketplace, showing their last updated date.Change taxonomy links from organization nodes to link to featured or all view.- Deploy changes on http://marketplace-drupal.redesign.devdrupal.org/ and update deployment instructions.
Deployment instructions:
- change url path for node/21791 from "drupal-services" to "drupal-services-old"
- remove redirect from "drupal-services" to "node/21791"
- change url path for node/313581 from "training-services" to "training-services-old"
- remove redirect from "training-services" to "node/313581"
- add redirect from /training-services to /training
- change link for Marketplace item in "Primary links" menu from "node/21791" to "drupal-services"
- update drupalorg, drupalorg.crossite modules and bluecheese theme
- update feature:
- CCK had "needs review" status, field group and content types had "overridden" status. Reverted everything to make it work.
- I'm not sure why Case studies feature has Marketplace as dependency
- existing organization nodes which are listed in the /marketplace-preview should get "field_organization_list_rule" set to "All providers"
- Arrange blocks:
- "Marketplace services menu" - place in Right column
- "Marketplace info" block - change title to "Marketplace info - all providers", place below services menu
content:All providers section lists companies which provide Drupal-related services and contribute back to the Drupal community. <a href="http://drupal.org/node/995766">Working with Drupal service providers</a> Forums: <a href="http://drupal.org/paid-services">Paid services</a>, <a href="http://drupal.org/hosting-support">Hosting support</a> Looking for <a href="http://groups.drupal.org/jobs">Jobs</a>? <a href="http://drupal.org/node/1735708">Marketplace guidelines</a> <a href="http://drupal.org/node/add/organization" class="action-button">Add your listing</a>Show only on:
drupal-services/all
drupal-services/all/* - block "Marketplace preview info" - change title to "Marketplace info - featured providers", place below services menu
content:Featured providers section lists companies which have an exceptional community contributions and show continued support of the Drupal project. <a href="http://drupal.org/node/995766">Working with Drupal service providers</a> Forums: <a href="http://drupal.org/paid-services">Paid services</a>, <a href="http://drupal.org/hosting-support">Hosting support</a> Looking for <a href="http://groups.drupal.org/jobs">Jobs</a>? <a href="http://drupal.org/node/1735708">Marketplace guidelines</a> <a href="http://drupal.org/node/add/organization" class="action-button">Add your listing</a>Show only on:
drupal-services
drupal-services/featured
drupal-services/featured/* - Add block "About trainings", place in Right sidebar
content:<a class="action-button" href="http://drupal.org/node/add/organization">Add your listing</a> <a href="http://drupal.org/node/1735708">Marketplace guidelines</a>Show only on:
training
training/* - block "drupalorg_training: Sidebar navigation" - place in Right column below "About trainings"
Show only on:
training
training/*
After deployment tasks:
- promote org nodes of the companies listed on /drupal-services-old to Featured providers
- add ogr nodes of companies listed on /training-services-old to Training section
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | bluecheese2.patch | 5.75 KB | tvn |
| #8 | drupalorg_crosssite2.patch | 3.82 KB | tvn |
| #7 | bluecheese.patch | 5.64 KB | tvn |
| #7 | drupalorg.patch | 13.6 KB | tvn |
| #7 | drupalorg_crosssite.patch | 3.82 KB | tvn |
Comments
Comment #1
tvn commentedworking on this on dev site.
Comment #1.0
tvn commentedadding instructions for not to forget later
Comment #1.1
tvn commented.
Comment #1.2
tvn commentedmore to-do
Comment #1.3
tvn commented.
Comment #1.4
tvn commented.
Comment #2
tvn commentedI've added new field to organization node type - field_organization_listing_type, with 2 values All and Featured. Restricted it to be admin-only via drupalorg module. Updated drupalorg_organizations view - added that field as an argument.
Old "Drupal services" book I temporarily moved to "Service old". It will be unpublished while we migrate organizations from there to the new marketplace.
Added menu block to the right sidebar on Marketplace and some descriptions of the types of listings - those texts are just examples and need improvement.
Lists of Categories on top of the Marketplace listings got broken a bit, drumm please take a look.
There is some number in () on the left and since I've added new argument to the view - urls of Categories need to be adjusted:
- column Services has correct urls "drupal-services/[featured|all]/[Service]"
- column Sectors has urls of the type "drupal-services/all/[featured|all]/[Sector]" - it should be "drupal-services/[featured|all]/all/[Sector]"
- column Countries served has urls of the type "drupal-services/all/all/[featured|all]/[Country]" - it should be "drupal-services/[featured|all]/all/all/[Country]"
Also maybe we should change url for "All providers"? It is now "all", same as wildcards you use for Categories.
Comment #3
tvn commented2 more things that still need work:
- page titles
- active trail for the menu on the right sidebar on subpages: featured/* and all/*
Also I'm going to add 1 more field to org node type - checkbox for Trainings listing and build a display for it.
Note that deployment instructions in the issue summary are not final yet, I'm just adding stuff on the go for not to forget it later.
Comment #3.0
tvn commentedadding link
Comment #3.1
tvn commentedadding remaining tasks
Comment #3.2
tvn commented.
Comment #3.3
tvn commentedadding more tasks
Comment #3.4
tvn commentedupdate
Comment #3.5
tvn commentedupdate
Comment #3.6
tvn commentedupdate
Comment #3.7
tvn commented.
Comment #4
drummI've been working on the 'tvn' dev site.
The view looks for both in filters. Additionally, field_organization_list_rule is used as an argument.
Comment #4.0
drumm.
Comment #5
drummI updated the issue summary:
Comment #6
tvn commented"Build a view of all org nodes listed in the marketplace, showing their last updated date." is done, added a display in drupalorg_training view.
Comment #6.0
tvn commentedUpdating todos
Comment #7
tvn commentedMarketplace feature attached, I added drupalorg_training view, new fields in org content type, menu "Service providers" and menu links.
Drupalorg, d_crosssite and bluecheese patches attached.
Going to update deployment instructions now.
Comment #7.0
tvn commented"view of all nodes" is done
Comment #7.1
tvn commented.
Comment #7.2
tvn commentedupdating instructions
Comment #7.3
tvn commentedupdate
Comment #7.4
tvn commented.
Comment #7.5
tvn commented.
Comment #7.6
tvn commented.
Comment #8
tvn commentedDeployed on "marketplace" dev site and updated deployment instructions.
Attached is updated bluecheese patch - block ids changed, and updated crosssite patch - added "/" when defining active tab in top menu (on /drupal-services-old both "Services" and "Service old" were active).
Comment #9
drummDeployed.
Organization nodes need to be found and featured for http://drupal.org/node/1649410. I did a few at the end out the alphabet.
Comment #10
jredding commentedA bug was found in the Marketplace. New Organization nodes are to be left unpublished and put into the webmaster's queue for moderation. They are currently auto-published and not placed in the webmaster's queue.
Comment #11
tvn commentedOrganization nodes should be published by default, but field_organization_list_rule default value should be "Do not list" instead of "All providers". I've changed it on the live site, we'll need to update this in the feature.
Comment #12
tvn commentedAnother minor thing - "All providers" section now shows both featured and not featured providers. Should it display only not featured ones?
Comment #13
drummI think "all" should show both featured & not. We don't ever want to hide featured providers since they have put the contributions in to be listed.
(If we have a listing of only non-featured, we should label it as "Other providers" or, more bluntly, "Non-featured providers.")
Comment #14
tvn commented@drumm, makes sense.
Comment #15
tvn commentedJust a note for future reference:
Between Aug17 (deployment) and Aug23 when #11 got fixed 6 org nodes were published and added to "All providers" section without review. I removed them from the listing so they would go through a proper review process:
http://drupal.org/node/1735164 The Web Development Group
http://drupal.org/node/1738596 DrupalSquad
http://drupal.org/node/1739862 INTERNET INTERNATIONAL TECHNOLOGY GROUP LIMITED
http://drupal.org/node/1741116 Vic Theme
http://drupal.org/node/1742954 Web Spiders
and
http://drupal.org/node/1742018 ThemeBrain
Comment #16
tvn commentedRecreated the feature over at #1760378: Improve Marketplace management view with correct default for field_organization_list_rule.
Comment #17
tvn commented#1760378: Improve Marketplace management view deployed. this is now fixed.
Comment #18.0
(not verified) commented.