Skip to content

Remove logout#100

Merged
rathishcholarajan merged 1 commit intoQiskit:mainfrom
jyu00:remove-logout
Jan 14, 2022
Merged

Remove logout#100
rathishcholarajan merged 1 commit intoQiskit:mainfrom
jyu00:remove-logout

Conversation

@jyu00
Copy link
Copy Markdown
Collaborator

@jyu00 jyu00 commented Jan 13, 2022

Summary

/logout as a public endpoint is being removed. This PR removes the corresponding code.

Details and comments

Fixes #

Copy link
Copy Markdown
Contributor

@daka1510 daka1510 left a comment

Choose a reason for hiding this comment

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

Looks good :). Glad it's no longer needed to remember calling logout for some reasons.

Copy link
Copy Markdown
Member

@rathishcholarajan rathishcholarajan left a comment

Choose a reason for hiding this comment

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

LGTM!

@rathishcholarajan rathishcholarajan merged commit 6be6b57 into Qiskit:main Jan 14, 2022
@rathishcholarajan rathishcholarajan added the Changelog: Removal Include in the Removed section of the changelog label Feb 24, 2022
blakejohnson pushed a commit to blakejohnson/qiskit-ibm-runtime that referenced this pull request May 26, 2023
* remove extra slash

* move find program id to utils
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changelog: Removal Include in the Removed section of the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants