-
-
Notifications
You must be signed in to change notification settings - Fork 71
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.
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
base: main
Are you sure you want to change the base?
Conversation
Hi @amvanbaren do you have the ability to test it? |
The report abuse link is supposed to stay the same? open-vsx.org/website/src/page-settings.tsx Lines 166 to 175 in 4edf5e1
|
It looks like a "space" is missing before the via and "marketplace", isn't it? |
@amvanbaren please change the "report abuse" email to [email protected] as well. |
Slight change of plans. The Infrastructure Team has added an alias to the inbox... Please change the email contact (in all locations) be |
cc @kineticsquid: do you see any place where [email protected] should be use (let appart here?) |
@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? |
Please, don't forget the "Report abuse" link on each extension detail that still links to [email protected] ![]() |
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])? |
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. |
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.