Skip to content

feat: Auto-populate page field when reporting an issue #1051

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

Merged
LadyBluenotes merged 43 commits into from
Mar 8, 2025
Merged

feat: Auto-populate page field when reporting an issue #1051

Changes from 7 commits
Commits
Show all changes
43 commits
Select commit Hold shift + click to select a range
0743679
fix: Make "Report an issue" link open in new tab
rmx90210 Jan 30, 2025
69a3f9a
chore: Clean up whitespace in page-issue-link classes
rmx90210 Jan 30, 2025
bf08606
feat: Auto-populate page field when reporting an issue
rmx90210 Jan 30, 2025
10124c2
Merge branch 'main' into iss-1049
kodiakhq[bot] Jan 30, 2025
059f2e5
Merge branch 'main' into iss-1049
kodiakhq[bot] Jan 30, 2025
be72882
Merge branch 'main' into iss-1049
kodiakhq[bot] Feb 6, 2025
1fada80
Merge branch 'main' into iss-1049
kodiakhq[bot] Feb 6, 2025
4e23f90
Merge branch 'main' into iss-1049
kodiakhq[bot] Feb 7, 2025
b77828b
Merge branch 'main' into iss-1049
kodiakhq[bot] Feb 7, 2025
740b7c8
Merge branch 'main' into iss-1049
kodiakhq[bot] Feb 7, 2025
71008c6
Merge branch 'main' into iss-1049
kodiakhq[bot] Feb 7, 2025
e209677
Merge branch 'main' into iss-1049
kodiakhq[bot] Feb 10, 2025
e4eaed9
Merge branch 'main' into iss-1049
kodiakhq[bot] Feb 13, 2025
8763095
Merge branch 'main' into iss-1049
kodiakhq[bot] Feb 13, 2025
da20136
Merge branch 'main' into iss-1049
kodiakhq[bot] Feb 13, 2025
f135ca3
Merge branch 'main' into iss-1049
kodiakhq[bot] Feb 13, 2025
ca19dd5
Merge branch 'main' into iss-1049
kodiakhq[bot] Feb 16, 2025
0ba2e65
Merge branch 'main' into iss-1049
kodiakhq[bot] Feb 16, 2025
6c6c386
Merge branch 'main' into iss-1049
kodiakhq[bot] Feb 16, 2025
49717c7
docs: resolve formatting issue in page-issue-link
rmx90210 Feb 16, 2025
691905c
Merge branch 'main' into iss-1049
kodiakhq[bot] Feb 18, 2025
4333236
Merge branch 'main' into iss-1049
kodiakhq[bot] Feb 18, 2025
793707c
Merge branch 'main' into iss-1049
kodiakhq[bot] Feb 18, 2025
0335938
Merge branch 'main' into iss-1049
kodiakhq[bot] Feb 18, 2025
38925ae
Merge branch 'main' into iss-1049
kodiakhq[bot] Feb 18, 2025
7a8de23
docs: use backticks in page-issue-link
rmx90210 Feb 18, 2025
bf27057
Update src/ui/page-issue-link.tsx
rmx90210 Feb 18, 2025
5e6e2c6
Merge branch 'main' into iss-1049
kodiakhq[bot] Feb 18, 2025
daa1ce1
Merge branch 'main' into iss-1049
kodiakhq[bot] Feb 18, 2025
a42d7b4
Merge branch 'main' into iss-1049
kodiakhq[bot] Feb 18, 2025
37efade
Merge branch 'main' into iss-1049
kodiakhq[bot] Feb 25, 2025
24f4bc8
Merge branch 'main' into iss-1049
kodiakhq[bot] Feb 25, 2025
137de91
Merge branch 'main' into iss-1049
kodiakhq[bot] Feb 26, 2025
146c042
Merge branch 'main' into iss-1049
kodiakhq[bot] Feb 26, 2025
c28f8f6
Merge branch 'main' into iss-1049
kodiakhq[bot] Mar 4, 2025
2c98c34
Merge branch 'main' into iss-1049
kodiakhq[bot] Mar 4, 2025
5d091c2
Merge branch 'main' into iss-1049
kodiakhq[bot] Mar 6, 2025
3dc4c93
Merge branch 'main' into iss-1049
kodiakhq[bot] Mar 7, 2025
942b37e
Merge branch 'main' into iss-1049
kodiakhq[bot] Mar 7, 2025
9be992b
Merge branch 'main' into iss-1049
kodiakhq[bot] Mar 7, 2025
4cec30c
Merge branch 'main' into iss-1049
kodiakhq[bot] Mar 7, 2025
660ad3b
Merge branch 'main' into iss-1049
kodiakhq[bot] Mar 7, 2025
1174ada
Merge branch 'main' into iss-1049
kodiakhq[bot] Mar 8, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 8 additions & 1 deletion src/ui/page-issue-link.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,14 @@ export const PageIssueLink: Component = () => {
const i18n = useI18n();

const srcPath = createMemo(() => {
return `https://github.com/solidjs/solid-docs-next/issues/new?assignees=ladybluenotes&labels=improve+documentation%2Cpending+review&projects=&template=CONTENT.yml&title=[Content]:&subject=${getEntryFileName()}`;
return `https://github.com/solidjs/solid-docs-next/issues/new` +
`?assignees=ladybluenotes` +
`&labels=improve+documentation%2Cpending+review` +
`&projects=` +
`&template=CONTENT.yml` +
`&title=[Content]:` +
`&subject=${getEntryFileName()}` +
`&page=${window.location.href}`
});
Copy link
Copy Markdown
Collaborator

@amirhhashemi amirhhashemi Feb 7, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

@bertday A formatting issue is blocking this PR. Could you please format the file or use the following code:

		return (
			"https://github.com/solidjs/solid-docs-next/issues/new" +
			"?assignees=ladybluenotes" +
			"&labels=improve+documentation%2Cpending+review" +
			"&projects=" +
			"&template=CONTENT.yml" +
			"&title=[Content]:" +
			"&subject=${getEntryFileName()}" +
			"&page=${window.location.href}"
		);

Copy link
Copy Markdown
Contributor Author

@rmx90210 rmx90210 Feb 16, 2025

Choose a reason for hiding this comment

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

Hi @amirhhashemi , this should be resolved now!

Copy link
Copy Markdown
Collaborator

@amirhhashemi amirhhashemi Feb 16, 2025

Choose a reason for hiding this comment

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

@bertday My bad, I should've used ` instead of " for strings. I didn't notice template strings are required.

Copy link
Copy Markdown
Contributor Author

@rmx90210 rmx90210 Feb 18, 2025

Choose a reason for hiding this comment

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

Hi @amirhhashemi , just pushed a change to bring this backticks back. Really hoping this does this trick.

return (
<a
Expand Down