Skip to content
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

Actualización de Links en Tablero Ingame #298

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

plusin
Copy link
Contributor

@plusin plusin commented Jan 26, 2024

Mergear cuando tengamos el dominio operativo, se actualizaron los links a los canales de Twitch y a la wiki con el nuevo dominio.-

We added new events in the GM Panel
We added new events in the GM Panel
Dejo esto para cuando usemos el dominio nuevo.-
@plusin
Copy link
Contributor Author

plusin commented Jan 26, 2024

Solo esto:

e23ecef

@RecoX
Copy link
Member

RecoX commented Feb 2, 2024

Muchos de estos links aun no estan en funcionamiento, cuando se termine la migracion del dominio argentumonline.com.ar se aprobara el PR.

@@ -260,7 +260,7 @@ Private Const WS_EX_LAYERED = &H80000
Private Const LWA_COLORKEY = &H1&

Private Sub BasicInfoLink_Click()
Call OpenLink("https://ao20.com.ar/wiki")
Call OpenLink("https://argentumonline.com.ar/wiki")
Copy link
Member

Choose a reason for hiding this comment

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

poner www en todos los links por favor.

Copy link
Contributor Author

@plusin plusin left a comment

Choose a reason for hiding this comment

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

Agregado www

Copy link

@codescene-delta-analysis codescene-delta-analysis bot left a comment

Choose a reason for hiding this comment

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

Code Health Quality Gates: OK

No scoreable code touched

View detailed results in CodeScene

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants