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

Assign key instead of unused comparison #5693

Merged
merged 1 commit into from
Jun 13, 2023

Conversation

dgrothaus-mc
Copy link
Contributor

Keys were compared without using the result.
Assign the key to the template instead.

@gharlan gharlan added this to the REDAXO 5.15.2 milestone May 17, 2023
@gharlan gharlan changed the base branch from main to bugfix May 17, 2023 20:22
@gharlan gharlan changed the base branch from bugfix to main May 17, 2023 20:24
@gharlan
Copy link
Member

gharlan commented May 17, 2023

@dgrothaus-mc Danke für den Fix! Ich muss den PR umswitchen, mit Zielbranch bugfix. Und dann rebasen.
Kannst du mir hier im PR Schreibrechte geben? Oder selbst umswitchen und rebasen?

Keys were compared without using the result.
Assign the key to the template instead.
@dgrothaus-mc dgrothaus-mc changed the base branch from main to bugfix May 23, 2023 06:29
@dgrothaus-mc
Copy link
Contributor Author

Hallo @gharlan.

Ich bin nicht sicher, ob ich richtig verstanden habe. Einen rebase auf bugfix habe ich jetzt gemacht.

Mit Rechte-Management bei GitHub kenne ich mich leider nicht ausreichend genug aus um hier in diesem PR Rechte vergeben zu können.

@gharlan
Copy link
Member

gharlan commented Jun 13, 2023

Mit Rechte-Management bei GitHub kenne ich mich leider nicht ausreichend genug aus um hier in diesem PR Rechte vergeben zu können.

Irgendwo in der rechten Seitenleiste müsste bei dir eine Checkbox sein "Allow edits from maintainers". Wobei der default eigentlich aktiviert ist. Evtl. war meine Vermutung auch falsch, dass er bei dir nicht aktiviert ist (ich kann das nicht genau sehen). Vielleicht lag es auch an irgendwas anderem, dass ich meine Änderungen nicht pushen konnte.

Wie auch immer, du hast das Rebasing ja selbst erledigt. 👍

@gharlan gharlan added the automerge Automatisch PR rebasen und mergen label Jun 13, 2023
@kodiakhq kodiakhq bot merged commit ef23d5d into redaxo:bugfix Jun 13, 2023
14 checks passed
@rex-bot rex-bot removed the automerge Automatisch PR rebasen und mergen label Jun 13, 2023
@dgrothaus-mc dgrothaus-mc deleted the template_key branch September 14, 2023 07:10
@gharlan gharlan modified the milestones: REDAXO 5.15.2, REDAXO 5.16 Jan 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants