Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DX] Convert /node to a View #162

Open
quicksketch opened this issue Jan 20, 2014 · 14 comments
Open

[DX] Convert /node to a View #162

quicksketch opened this issue Jan 20, 2014 · 14 comments

Comments

@quicksketch
Copy link
Member

Sub-issue of #151. We should convert the main listing provided by node module to a view.

Drupal.org issue: https://drupal.org/node/1806334

@DirectorHaas
Copy link

Just checked this out. Simple VBO seems to be working. Question: so what we want here is the exported to code versions of all the Views that have issues opened, yes?

Seems straightforward enough, so is the status of this still good for someone to do the Views click around and export these to custom views code?

Thinking the answer is yes, getting verification before spending evening / weekend time. Thank you!

@DirectorHaas
Copy link

Also looks like VBO are primarily for Content, not user (or taxonomy?)

" Content: Bulk operations
Provides a checkbox on each row and a form for executing bulk actions on nodes."

So something like add relationship:
"RELATIONSHIPS
User: Representative node"

Will need to get a little creative w/ it? Asking again if were 99% ready to move on the views and any special format we need to have the exported views/custom code. Thanks!

@quicksketch
Copy link
Member Author

Also looks like VBO are primarily for Content, not user (or taxonomy?)

For now we're looking at converting admin/content (#150) and admin/people (#149) to views. Both nodes and users should be VBO-capable at this point.

Asking again if were 99% ready to move on the views and any special format we need to have the exported views/custom code.

For this issue, it's pretty straight-forward I think. When you export the view using the Config Manager, it'll export it into a JSON file. Just drop this file in /core/modules/node/config. After exporting, you'll need to manually open the file and add two lines to the JSON file to make it act as a "default" config:

  "storage": 4,
  "module": "node",

This will prevent the /node view from being possible to delete (though you can still disable it) and makes the "revert" option show up in the Views interface. See the config files in /core/modules/image/config for examples.

@DirectorHaas
Copy link

Thanks for the info. Noticed everything is really shaping up.

When it's fairly stable, interested looking into converting contrib modules like Feeds, maybe Wordpress_Migrate, and also looking at best ways to get D6 / D7 data into Backdrop.

May be wishful thinking but, hey, why not.

@DirectorHaas
Copy link

Spending some evening w/ this view "Issue #150: Convert admin/content to a View." Things seem straightforward except for the drop-down in the last col for EDIT/DELETE. Maybe overlooking something there, trying to use a Global: drop-button, may need to hack the exported views code to put in an edit and delete dropdown.

More importantly, and perhaps it's just a permission, but not seeing a "Views" to "Export" option or did it move around where it's not to be seen! Heh.

Thank you.

@DirectorHaas
Copy link

I should have commented this in the other thread, since it is about admin/content instead of admin/node, but the Views->Export function could be useful for both, may just be hidden?

@quicksketch
Copy link
Member Author

The pages for exporting directly within Views were removed because just about everything is exportable now. Rather than making separate interfaces, all exporting/importing for the entire site is done at the Configuration management interface. Although it lacks an individual export, you can download the entire zip and then pull out the config you want. Although it's generally easier just to pluck the config file you want out of the files/config_*/active directory. The concept of "exporting" generally is dead, since all configuration is always in an exported form to start with. You just need to copy the export to a module's config directory to have it imported on module install.

@DirectorHaas
Copy link

Just tried a test, neat! I should have been following everything more closely, been working w/ Apps, not Drupal-y stuff for much of last year. Pretty cool. Thanks again!

@tarekdj
Copy link
Member

tarekdj commented Jan 13, 2015

PR: backdrop/backdrop#656
Ready!

@jenlampton jenlampton changed the title Convert /node to a View [DX] Convert /node to a View Jan 3, 2016
@klonos
Copy link
Member

klonos commented Nov 25, 2018

Is this issue here still relevant? /node currently redirects you to the home page, and AFIK we have a layout and a default view for it.

@laryn
Copy link
Contributor

laryn commented Nov 26, 2018

@klonos You're right. See @docwilmot comment in the old PR: backdrop/backdrop#656 (comment)

@klonos
Copy link
Member

klonos commented Nov 26, 2018

...yeah, lets close it then 😄

@klonos klonos closed this as completed Nov 26, 2018
@klonos
Copy link
Member

klonos commented Jan 3, 2022

While testing the PR for #3343, we've accidentally discovered that this was never actually done. The promoted view provided by Backdrop core only provides a block and a feed display, whereas there should be a page display as well (which should have /node for its path).

We should also adjust node_page_default() to use a views_embed_view() + call watchdog_deprecated_function(), and remove it later in 2.x.

@izmeez
Copy link

izmeez commented Jan 30, 2024

It's not clear if this was finalized in another issue or remains open: replace /node with a view. The last few comments suggest it was agreed upon and then forgotten. There's an old PR and the patch only applies partially and will need work if this issue is still alive.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants