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.

Sign up for GitHub

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

Jump to bottom

Bug: Handle the double slashes in the url for tweets #42

Merged
merged 1 commit into from
Jan 4, 2021
Merged

Bug: Handle the double slashes in the url for tweets #42

merged 1 commit into from
Jan 4, 2021

Conversation

g8rswimmer
Copy link
Owner

for the following endpoints, there have been double slashes in the url

  • tweet filtered stream rules
  • tweet filtered stram
  • tweet sampled stream
  • tweet hide

Issue #41

@g8rswimmer g8rswimmer added bug Something isn't working v1 Version 1 labels Jan 4, 2021
@g8rswimmer g8rswimmer added this to the v1.1.2 milestone Jan 4, 2021
@g8rswimmer g8rswimmer self-assigned this Jan 4, 2021
@g8rswimmer g8rswimmer linked an issue Jan 4, 2021 that may be closed by this pull request
Handle the double backslashes #41
Closed
Guitarbum722
Guitarbum722 approved these changes Jan 4, 2021
View reviewed changes
Copy link
Collaborator

@Guitarbum722 Guitarbum722 left a comment

Choose a reason for hiding this comment

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

LGTM

@g8rswimmer g8rswimmer merged commit 3a4a858 into master Jan 4, 2021
@g8rswimmer g8rswimmer deleted the bugfix/issue-41-double-backslash branch January 4, 2021 02:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Guitarbum722 Guitarbum722 Guitarbum722 approved these changes

Assignees

@g8rswimmer g8rswimmer

Labels
bug Something isn't working v1 Version 1
Projects
None yet
Milestone
v1.1.2
Development

Successfully merging this pull request may close these issues.

Handle the double backslashes
2 participants
@g8rswimmer @Guitarbum722