Skip to content

Hotfix for internal links to replay files not working (#42)#42

Merged
DevGeniusCode merged 1 commit intoTheSuperHackers:mainfrom
Skyaero42:hotfix/replay-urls-not-working
Apr 29, 2025
Merged

Hotfix for internal links to replay files not working (#42)#42
DevGeniusCode merged 1 commit intoTheSuperHackers:mainfrom
Skyaero42:hotfix/replay-urls-not-working

Conversation

@Skyaero42
Copy link
Contributor

@Skyaero42 Skyaero42 commented Apr 26, 2025

Relative internal links to the replay files don't work. This hotfix changes the links into absolute urls point directly at the source code.

Skyaero42 pushed a commit to Skyaero42/GeneralsWiki that referenced this pull request Apr 26, 2025
@Skyaero42 Skyaero42 force-pushed the hotfix/replay-urls-not-working branch from 30254e3 to 3dbf112 Compare April 26, 2025 10:19
@DevGeniusCode
Copy link
Contributor

Can you add a hotfix to remove the Debug folder from the file path on the home and sidebar pages?

@Skyaero42
Copy link
Contributor Author

Can you add a hotfix to remove the Debug folder from the file path on the home and sidebar pages?

I'm not entirely sure what you mean. Do you want me to move the replay_testing.md from Debug folder to the main folder?

@DevGeniusCode
Copy link
Contributor

No, just remove from the file path, the link to the file should only be the file name

Skyaero42 pushed a commit to Skyaero42/GeneralsWiki that referenced this pull request Apr 27, 2025
@Skyaero42 Skyaero42 force-pushed the hotfix/replay-urls-not-working branch from 3dbf112 to 6c05630 Compare April 27, 2025 07:52
@Skyaero42
Copy link
Contributor Author

Added to this hotfix.

@Skyaero42 Skyaero42 force-pushed the hotfix/replay-urls-not-working branch from 6c05630 to 56b1e9b Compare April 27, 2025 09:18
@DevGeniusCode DevGeniusCode merged commit 0dfd388 into TheSuperHackers:main Apr 29, 2025
1 check passed
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

Successfully merging this pull request may close these issues.

2 participants