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
Fix: st.page_link
URL preview shows file path
#8086
Conversation
@@ -682,7 +682,7 @@ def _page_link( | |||
if label is None: | |||
page_link_proto.label = page_name | |||
page_link_proto.page_script_hash = page_data["page_script_hash"] | |||
page_link_proto.page = full_path |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
page is fed to the href
attribute in PageLink.tsx
- this is what shows in the browser preview in the bottom left corner.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, although I think we'll also want to verify the fields are all set correctly in a unit test if it's not too difficult to add one
Fix href setting for page links and update error message on invalid pages
Fix href setting for page links and update error message on invalid pages
Describe your changes
Fix href setting for page links
GitHub Issue Link (if applicable)
Closes #8080