Skip to content

Rename createPresentationTokenContent#936

Merged
Milena-Czierlinski merged 3 commits intorelease/openid4vcfrom
consistent-naming-create-presentation-token
Jan 23, 2026
Merged

Rename createPresentationTokenContent#936
Milena-Czierlinski merged 3 commits intorelease/openid4vcfrom
consistent-naming-create-presentation-token

Conversation

@Milena-Czierlinski
Copy link
Member

@Milena-Czierlinski Milena-Czierlinski commented Jan 23, 2026

Readiness checklist

  • I added/updated tests.
  • I ensured that the PR title is good enough for the changelog.
  • I labeled the PR.
  • I self-reviewed the PR.

@Milena-Czierlinski Milena-Czierlinski added the refactoring Refactoring of code label Jan 23, 2026
@Milena-Czierlinski Milena-Czierlinski marked this pull request as ready for review January 23, 2026 08:36
Copy link
Contributor

@Magnus-Kuhn Magnus-Kuhn left a comment

Choose a reason for hiding this comment

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

Those don't create actual tokens though. I'd be fine with something like createPresentationTokenContent or createPresentationForPresentationToken though - is that acceptable?

@Milena-Czierlinski
Copy link
Member Author

Alright, I'll change it to createPresentationTokenContent. 👍

@Milena-Czierlinski Milena-Czierlinski changed the title Use naming createPresentationToken consistently Rename createPresentationTokenContent Jan 23, 2026
@Milena-Czierlinski Milena-Czierlinski enabled auto-merge (squash) January 23, 2026 09:16
@Milena-Czierlinski Milena-Czierlinski merged commit 680fc81 into release/openid4vc Jan 23, 2026
24 of 25 checks passed
@Milena-Czierlinski Milena-Czierlinski deleted the consistent-naming-create-presentation-token branch January 23, 2026 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactoring Refactoring of code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants