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

Removed SessionIdentifierAwareInterface #62

Closed
wants to merge 2 commits into from

Conversation

alexmerlin
Copy link
Member

Q A
Documentation yes
Bugfix no
BC Break yes
New Feature no
RFC no
QA no

Description

As requested in #59 this commit:

  • adds SessionIdentifierAwareInterface's getId() method to SessionInterface
  • removes SessionIdentifierAwareInterface

@alexmerlin
Copy link
Member Author

@gsteel Here are the changes for the 2.0.x branch.

Copy link
Member

@gsteel gsteel left a comment

Choose a reason for hiding this comment

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

Thanks @alexmerlin - I should have mentioned that we'll need to get the docs setup for the v1/v2 split before we can start merging patches that require docs updates.

We'll also need to setup a migration guide too.

Signed-off-by: alexmerlin <[email protected]>
@gsteel
Copy link
Member

gsteel commented Aug 19, 2024

There's something weird going on here @alexmerlin - it'll be easier if you send another PR to first duplicate the v1 docs to v2 without changing anything, then, rebase this PR on to the tip of 2.0.x, re-applying the changes to the v2 docs tree.

@gsteel gsteel removed this from the 2.0.0 milestone Aug 19, 2024
@gsteel gsteel self-assigned this Aug 19, 2024
@gsteel gsteel closed this Aug 19, 2024
@alexmerlin alexmerlin deleted the issue-59 branch August 21, 2024 09:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants