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

Update static links to meetup website #27

Merged
merged 3 commits into from
Jan 27, 2023
Merged

Conversation

2ndkauboy
Copy link
Contributor

Closes #20

@2ndkauboy 2ndkauboy force-pushed the fix/20-update-links branch from 99ae51c to e7f5ab8 Compare October 25, 2022 06:50
@Zodiac1978
Copy link
Member

LGTM.

Da ich jetzt über 3 Wochen keine Antwort von @krafit auf Slack bekommen habe (trotz mehrfacher Nachfragen), wie der Status ist, und ob er die API reparieren kann, macht es wahrscheinlich Sinn, das hier zu mergen, um mal die kaputten Links auf den Meetup-Websites zu fixen.

Zodiac1978
Zodiac1978 previously approved these changes Nov 2, 2022
@Zodiac1978
Copy link
Member

Zodiac1978 commented Nov 2, 2022

Bin nochmal alle Links durchgegangen und doch noch zwei Probleme gefunden:
https://wpmeetup-karlsruhe.de/ (Fatal Error)
https://wpmeetup-bremen.de/ (Zertifikatsfehler, leitet aber von http auf https)

@Zodiac1978 Zodiac1978 dismissed their stale review November 2, 2022 10:14

Zwei Links sind noch kaputt

@Zodiac1978
Copy link
Member

Habe beide Meetups mal über den Slack-Channel angeschrieben.

@Zodiac1978
Copy link
Member

Rückmeldung aus Bremen: Erstmal den Meetup-Link nehmen:
https://www.meetup.com/de-DE/bremen-wordpress-meetup-group/

PHP CodeBeautifier applied, fixed outdated comments and added missing function visibility.
@Zodiac1978
Copy link
Member

Karlsruhe läuft wieder

Copy link
Member

@Zodiac1978 Zodiac1978 left a comment

Choose a reason for hiding this comment

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

LGTM

@2ndkauboy 2ndkauboy merged commit 38017cf into stable Jan 27, 2023
@2ndkauboy 2ndkauboy deleted the fix/20-update-links branch January 27, 2023 21:29
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.

Kaputte Links im Widget
2 participants