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

Unable to access json/xml/jsonld via rest #392

Closed
Natkeeran opened this issue Oct 5, 2016 · 15 comments
Closed

Unable to access json/xml/jsonld via rest #392

Natkeeran opened this issue Oct 5, 2016 · 15 comments
Assignees

Comments

@Natkeeran
Copy link
Contributor

Natkeeran commented Oct 5, 2016

Steps to reproduce:

Expected behavior:

The actual, incorrect behavior:

@whikloj whikloj self-assigned this Oct 5, 2016
@whikloj
Copy link
Member

whikloj commented Oct 5, 2016

This seems to be related to the drupal version. I modified the vagrant to install Drupal 8.1.8 and I could once again see the permissions entries for the various REST endpoints.

@ruebot
Copy link
Member

ruebot commented Oct 5, 2016

...and it looks like 8.2.0 was released today. https://www.drupal.org/blog/drupal-8-2-0

So, that should explain why 8.2.0 is getting installed. I was confused for a moment 😄

@whikloj
Copy link
Member

whikloj commented Oct 5, 2016

And RESTUI is not ready for it.
https://www.drupal.org/node/2758563

@DiegoPino
Copy link
Contributor

@whikloj @ruebot if RESTUI is not ready (and 8.2 is different, some libraries changed) REST API can be set using configuration YAMLl file. Do you want me to do that?

@whikloj
Copy link
Member

whikloj commented Oct 6, 2016

IMHO that is just going to make it harder for anyone to learn how to use. So if you all want to do that, I wouldn't argue. But I think we should just move back to 8.1.8 and wait. Then people can see how it should work.

@DiegoPino
Copy link
Contributor

@whikloj ok i agree. 8.1.8 until A) RESTUI gets an update or B) we update RESTUI!

@Natkeeran
Copy link
Contributor Author

@whikloj @ruebot To avoid issues such as this in the future, should we always target a specific version of drupal and other major components for vagrant? Or Is it ok since this is the development environment and we want to find out potential issues at the earliest?

@whikloj
Copy link
Member

whikloj commented Oct 6, 2016

@Natkeeran hard to say, if the plan is that these permissions are disappearing for good. Then we should switch and configure as @DiegoPino suggests. But if not then we just seem to be a little ahead of the curve and waiting until they get an update completed makes the most sense.

That being said, perhaps we should try using the dev version.

@DiegoPino
Copy link
Contributor

@Natkeeran yes for Drupal. The HUGE difference with Drupal 8 is that changes between mayor versions are deprecating/replacing the guts of it. So 8.2 is really different that 8.1. That did not happen in Drupal 7.x which since was mostly procedural. So we just need more experience with Drupal.

@dannylamb
Copy link
Contributor

dannylamb commented Oct 6, 2016

I say we always target a specific version. Best to be explicit.

And I'd wait on everything else to catch up and we deal with 8.2 when it's just our stuff that needs to be updated.

@dannylamb
Copy link
Contributor

#399 won't fix restui, but it will keep us from getting suprised by a new version. Can someone check it out?

@dannylamb
Copy link
Contributor

I'm running a fresh vagrant up to test the install of a fix that upgrades us to Drupal 8.2.2 and installs a rest config entity for fedora resources. I'll keep everyone posted.

@dannylamb
Copy link
Contributor

LOL, and while waiting for my vagrant up to go.... the REST UI module pushed a release for 8.2!

@dannylamb
Copy link
Contributor

Bump #416

I need a test and merge.

@ruebot
Copy link
Member

ruebot commented Nov 16, 2016

Resolved with: 0d96627

@ruebot ruebot closed this as completed Nov 16, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants