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

Moved 'contributing' images to their own directory #4200

Merged
merged 1 commit into from
Sep 15, 2014
Merged

Moved 'contributing' images to their own directory #4200

merged 1 commit into from
Sep 15, 2014

Conversation

javiereguiluz
Copy link
Member

Q A
Doc fix? yes
New docs? no
Applies to 2.3+
Fixed tickets #3192

The PDF version of the "Contributing Guide" doesn't show the images (as reported in #3192). The rest of the PDFs correctly show them. Although I don't know the exact cause of the error, the "Contributing" guide images weren't stored in their own subdirectory inside images/ and named after the book. That's the only difference between the right and the wrong PDF files.

For compatibility reasons I've left on purpose the original images at their location and just made a copy in the new directory.

@stof
Copy link
Member

stof commented Sep 5, 2014

I suggest adding a small note in the doc about contributing to the documentation saying that images should not be placed in images directly, to avoid such error in the future

@wouterj
Copy link
Member

wouterj commented Sep 8, 2014

@stof let's first find out if this solves the issue.

@weaverryan
Copy link
Member

Ok, let's see what happens!

@weaverryan weaverryan merged commit fc15b65 into symfony:2.3 Sep 15, 2014
weaverryan added a commit that referenced this pull request Sep 15, 2014
…reguiluz)

This PR was merged into the 2.3 branch.

Discussion
----------

Moved 'contributing' images to their own directory

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies to    | 2.3+
| Fixed tickets | #3192

The PDF version of the "Contributing Guide" doesn't show the images (as reported in #3192). The rest of the PDFs correctly show them. Although I don't know the exact cause of the error, the "Contributing" guide images weren't stored in their own subdirectory inside `images/` and named after the book. That's the only difference between the right and the wrong PDF files.

For compatibility reasons I've left on purpose the original images at their location and just made a copy in the new directory.

Commits
-------

fc15b65 Moved 'contributing' images to their own directory
@xabbuh
Copy link
Member

xabbuh commented Sep 16, 2014

This didn't seem to fix the issue. :( The images are still not included (and yes, the PDF has been regenerated after the merge of this pull request).

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

Successfully merging this pull request may close these issues.

5 participants