Skip to content

fix: Always clear cache when on MCP page to help with redirect issue #1801

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 8 commits into from
Aug 22, 2025
Merged

fix: Always clear cache when on MCP page to help with redirect issue #1801

merged 8 commits into from
Aug 22, 2025

Conversation

jirispilka
Copy link
Contributor

@jirispilka jirispilka commented Aug 15, 2025 •
edited
Loading

Always clear cache when on MCP page to help with redirect issues.
Previously, mcp.apify.com had a 301 redirect to this docs page, which was wrong.
Many users have this redirect cached in their browsers, so they can't access the new MCP UI.
This helps with their cached redirect, allowing them to access mcp.apify.com

@B4nan is there is a better place to put the code than in the src/theme/layout?

Kudos to @JJetmar

FYI @MQ37

close apify/apify-mcp-server#129
close https://github.com/apify/ai-team/issues/33

EDIT: Remove this fix after some time: #1802

@jirispilka jirispilka requested a review from TC-MO as a code owner August 15, 2025 10:17
@github-actions github-actions bot added the t-ai Issues owned by the AI team. label Aug 15, 2025
@jirispilka jirispilka changed the title fix: Always clear cache when on MCP page to help with redirect issues fix: Always clear cache when on MCP page to help with redirect issue Aug 15, 2025
@apify-service-account
Copy link

Preview for this PR was built for commit 1e99900 and is ready at https://pr-1801.preview.docs.apify.com!

TC-MO
TC-MO approved these changes Aug 15, 2025
View reviewed changes
Copy link
Contributor

@TC-MO TC-MO left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM (tho not that great at React :D)

@jirispilka jirispilka requested a review from B4nan August 15, 2025 10:22
@apify-service-account
Copy link

Preview for this PR was built for commit 7b9a414 and is ready at https://pr-1801.preview.docs.apify.com!

@jirispilka jirispilka mentioned this pull request Aug 15, 2025
Open
@jirispilka
Copy link
Contributor Author

@TC-MO I made a small update to the text. We would like to gently encourage people to star the GitHub page.

I tried adding a badge with the star count, but it did not look great, so I decided to include it as plain text instead.

**Open source**—You can [raise issues](https://github.com/apify/actors-mcp-server/issues) or [suggest new features](https://github.com/apify/actors-mcp-server/issues/new). If you find this useful, please [star us on GitHub](https://github.com/apify/actors-mcp-server) to show your support!

@jirispilka jirispilka requested a review from TC-MO August 18, 2025 06:25
@apify-service-account
Copy link

Preview for this PR was built for commit 09b9d7af and is ready at https://pr-1801.preview.docs.apify.com!

TC-MO
TC-MO requested changes Aug 18, 2025
View reviewed changes
Copy link
Contributor

@TC-MO TC-MO left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some changes suggested, but also broader discussion about placement of both notes

@jirispilka @TC-MO
Update sources/platform/integrations/ai/mcp.md
82fadea 
Co-authored-by: Michał Olender <92638966+TC-MO@users.noreply.github.com>
@apify-service-account
Copy link

Preview for this PR was built for commit 82fadeaa and is ready at https://pr-1801.preview.docs.apify.com!

@jirispilka
Copy link
Contributor Author

jirispilka commented Aug 18, 2025 •
edited
Loading

@TC-MO I took care of your comments, but I believe, once you'll revisit the document, you might change it anyway (to find a better place)

@apify-service-account
Copy link

Preview for this PR was built for commit 42b7d9a1 and is ready at https://pr-1801.preview.docs.apify.com!

B4nan
B4nan approved these changes Aug 19, 2025
View reviewed changes
Copy link
Member

@B4nan B4nan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess I don't mind where the script is implemented.

Alternatively, we have this JS script added to all the pages, where we do all sorts of random JS quirks (outside of react):

https://github.com/apify/apify-docs/blob/master/apify-docs-theme/static/js/custom.js

TC-MO
TC-MO approved these changes Aug 19, 2025
View reviewed changes
Copy link
Contributor

@TC-MO TC-MO left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

small visual nit: I think it makes sense to use emphasis with italics within the quick start note otherwise, LGTM

@jirispilka @TC-MO
Apply suggestions from code review
e6f8945 
Co-authored-by: Michał Olender <92638966+TC-MO@users.noreply.github.com>
@apify-service-account
Copy link

Preview for this PR was built for commit e6f89456 and is ready at https://pr-1801.preview.docs.apify.com!

@apify-service-account
Copy link

Preview for this PR was built for commit 74c77d85 and is ready at https://pr-1801.preview.docs.apify.com!

@jirispilka
Copy link
Contributor Author

@B4nan thanks. I've moved to custom.js, make sense.

If you can, please give a quick look and we can release it.
I've tested it at local, it works

@jirispilka jirispilka requested a review from B4nan August 19, 2025 15:35
@jirispilka
Copy link
Contributor Author

Let's merge this and see: #yolo

@jirispilka jirispilka merged commit 4451c5e into master Aug 22, 2025
10 checks passed
@jirispilka jirispilka deleted the fix/add-link-ui-redirect branch August 22, 2025 09:56
@MQ37
Copy link
Contributor

MQ37 commented Aug 22, 2025 •
edited
Loading

Let's merge this and see: #yolo

based 😎

daveomri pushed a commit to daveomri/apify-docs that referenced this pull request Sep 3, 2025
@jirispilka @TC-MO @daveomri
fix: Always clear cache when on MCP page to help with redirect issue (a…
903acbb 
…pify#1801)

Always clear cache when on MCP page to help with redirect issues.
Previously, mcp.apify.com had a 301 redirect to this docs page, which
was wrong.
Many users have this redirect cached in their browsers, so they can't
access the new MCP UI.
This helps with their cached redirect, allowing them to access
mcp.apify.com

@B4nan is there is a better place to put the code than in the
src/theme/layout?

Kudos to @JJetmar 

FYI @MQ37 

close apify/apify-mcp-server#129
close https://github.com/apify/ai-team/issues/33

EDIT: Remove this fix after some time:
apify#1802

---------

Co-authored-by: Michał Olender <92638966+TC-MO@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@TC-MO TC-MO TC-MO approved these changes

@B4nan B4nan Awaiting requested review from B4nan

Assignees

@jirispilka jirispilka

Labels

t-ai Issues owned by the AI team.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@jirispilka @apify-service-account @MQ37 @B4nan @TC-MO