Skip to content

FIX: Broken 'About' Link in the Footer#23367

Merged
mdo merged 1 commit intotwbs:v4-devfrom
ahmadawais:patch-3
Aug 15, 2017
Merged

FIX: Broken 'About' Link in the Footer#23367
mdo merged 1 commit intotwbs:v4-devfrom
ahmadawais:patch-3

Conversation

@ahmadawais
Copy link
Contributor

The About link in the footer is broken. This PR fixes that.

gif

Hopefully, this will be my first contribution to the Bootstrap project.

The `About` link in the footer is broken. This PR fixes that.

![gif](https://i.imgur.com/3H4E04X.gif)

Hopefully, this will be my first contribution to _the_ Bootstrap project.
@ahmadawais
Copy link
Contributor Author

@XhmikosR Apologies for the tag, but this is a similar PR to the last one I submitted. The about link is also broken.

@XhmikosR
Copy link
Member

XhmikosR commented Aug 12, 2017 via email

@ahmadawais
Copy link
Contributor Author

@XhmikosR I tried. I didn't work. Not sure why! But this one is the similar PR to the Example's broken link. At least it fixes the gh-pages branch — for now.

@XhmikosR
Copy link
Member

XhmikosR commented Aug 12, 2017 via email

@ahmadawais
Copy link
Contributor Author

I am not sure. Have not seen what you guys are using to setup custom routing. First time here to contribute something. Been a regular core contributor at WordPress for years.

@XhmikosR
Copy link
Member

XhmikosR commented Aug 12, 2017 via email

@mdo
Copy link
Member

mdo commented Aug 12, 2017

I probably have a redirect on the live docs site that's interfering with anything we do in thev4-dev branch for that situation. If it works locally though, we should be okay.

@ahmadawais
Copy link
Contributor Author

I tried but I guess I wasn't able to build it correctly. Or maybe I did Though, it mimics the behavior which is here in the production.

@XhmikosR
Copy link
Member

@mdo: how do you update the gh_pages branch? Just build from v4-dev and overwrite everything? If not, can you push any of your stuff that are not in v4-dev there so that we are in sync?

And close/merge this, although I merged the examples patch even though locally it worked for me as it was.

@mdo
Copy link
Member

mdo commented Aug 13, 2017

@XhmikosR I manually was updating some things, but I don't regularly push from v4-dev. Hosted docs don't stay up to date with latest dev branch, they stay with the release.

@mdo mdo merged commit 0909b46 into twbs:v4-dev Aug 15, 2017
@mdo mdo mentioned this pull request Aug 15, 2017
@ahmadawais ahmadawais deleted the patch-3 branch April 28, 2021 15:18
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.

4 participants