Skip to content

Add email contact information. #4577

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 1 commit into from
Aug 18, 2025
Merged

Add email contact information. #4577

merged 1 commit into from
Aug 18, 2025

Conversation

waynebeaton
Copy link
Contributor

This patch adds a pointer to the [email protected] inbox.

I don't have a means of testing this, so I'm only mostly sure that it's correct. This is further complicated by the fact that I am unfamiliar with the specific details of the markup language used, so I've made what I believe are really good guesses at what should be there.

Please test this before merging.

@tfroment
Copy link
Contributor

Hi @amvanbaren do you have the ability to test it?

@amvanbaren
Copy link
Contributor

This is what it looks like:
image

@amvanbaren
Copy link
Contributor

The report abuse link is supposed to stay the same?

open-vsx.org/website/src/page-settings.tsx

Lines 166 to 175 in 4edf5e1

//---------- REPORT ABUSE LINK
const reportAbuse: FunctionComponent<{ extension: Extension, sx: SxProps<Theme> }> = ({ extension, sx }) => {
const reportAbuseText = encodeURIComponent('<Please describe the issue>');
const extensionURL = encodeURIComponent(`${location.protocol}//${location.hostname}/extension/${extension.namespace}/${extension.name}`);
return <Link
href={`mailto:[email protected]?subject=Report%20Abuse%20-%20${extension.namespace}.${extension.name}&Body=${reportAbuseText}%0A%0A${extensionURL}`}
variant='body2' color='secondary' underline='hover' sx={sx} >
Report Abuse
</Link>;
};

@tfroment
Copy link
Contributor

tfroment commented Aug 4, 2025 •
edited
Loading

This is what it looks like

It looks like a "space" is missing before the via and "marketplace", isn't it?

@waynebeaton
Copy link
Contributor Author

@amvanbaren please change the "report abuse" email to [email protected] as well.

@waynebeaton
Copy link
Contributor Author

Slight change of plans. The Infrastructure Team has added an alias to the inbox...

Please change the email contact (in all locations) be [email protected]

@tfroment
Copy link
Contributor

tfroment commented Aug 5, 2025

cc @kineticsquid: do you see any place where [email protected] should be use (let appart here?)

@kineticsquid
Copy link
Contributor

@tfroment I could imagine including the 'report abuse' text and email link also on the 'about' page.

@tfroment
Copy link
Contributor

tfroment commented Aug 7, 2025

@tfroment I could imagine including the 'report abuse' text and email link also on the 'about' page.

Hi @kineticsquid, yes would be great, can you contribute to this?

@mbarbero
Copy link
Member

Please, don't forget the "Report abuse" link on each extension detail that still links to [email protected]

image

@tfroment tfroment requested a review from kineticsquid August 11, 2025 09:39
@kineticsquid
Copy link
Contributor

Want to make sure we're in synch on this. We have an email link in two places, on the 'Report Abuse' link on each extension page and on the 'About' page under 'Contact Us'. Should these links be [email protected] or [email protected] (or [email protected])?

@kineticsquid
Copy link
Contributor

@mbarbero @waynebeaton ?

@tfroment
Copy link
Contributor

@mbarbero @waynebeaton ?

Hi @kineticsquid, we should use [email protected] (it’s an alias of [email protected]), but it’s better to make openvsx visible in all the places mentioned in this issue.

kineticsquid
kineticsquid requested changes Aug 18, 2025
View reviewed changes
Copy link
Contributor

@kineticsquid kineticsquid left a comment

Choose a reason for hiding this comment

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

Change the email address to [email protected]. I'd also make a slight adjustments to the text that follows:

        For other non-urgent questions, including managing namespaces or publishing extensions, please visit the 
        <Link color='secondary' underline='hover' href='https://github.com/EclipseFdn/open-vsx.org/wiki'>wiki</Link>

@kineticsquid
Copy link
Contributor

I was attempting to submit a PR to change the email address at the 'Report Abuse' link in page-settings.tsx. When I saved my changes, instead of being prompted to submit PR, I ended up editing the file directly. Seems I still have write access. Anyway, I'll have to remember that in the future.

open-vsx.org/website/src/page-settings.tsx

Line 171 in 70e7cc5

href={`mailto:[email protected]?subject=Report%20Abuse%20-%20${extension.namespace}.${extension.name}&Body=${reportAbuseText}%0A%0A${extensionURL}`}

@kineticsquid kineticsquid merged commit 458c558 into EclipseFdn:main Aug 18, 2025
3 checks passed
@tfroment
Copy link
Contributor

Hi @kineticsquid @amvanbaren; do you know when this will go into production? We need this asap, Thanks!

@kineticsquid
Copy link
Contributor

Production PR: #4884 (review)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kineticsquid kineticsquid kineticsquid requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@waynebeaton @tfroment @amvanbaren @kineticsquid @mbarbero