Skip to content

Replace http with https in the About window#3572

Merged
justinclift merged 5 commits into
sqlitebrowser:masterfrom
rafrafek:master
Feb 23, 2024
Merged

Replace http with https in the About window#3572
justinclift merged 5 commits into
sqlitebrowser:masterfrom
rafrafek:master

Conversation

@rafrafek

@rafrafek rafrafek commented Feb 22, 2024

Copy link
Copy Markdown
Contributor

Fixes #3571

@justinclift

Copy link
Copy Markdown
Member

Heh Heh Heh

I forgot about all the .ts files too. Thanks for including them in the change. 😄

@justinclift

Copy link
Copy Markdown
Member

Please ping me when you feel this is ready, as I'm not sure if GitHub will alert me otherwise. 😄

@rafrafek rafrafek marked this pull request as ready for review February 23, 2024 01:41
@rafrafek

Copy link
Copy Markdown
Contributor Author

@justinclift, I think it's ready!

@justinclift justinclift left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good to me. 😄

@justinclift justinclift merged commit fdaf1ce into sqlitebrowser:master Feb 23, 2024
@justinclift

Copy link
Copy Markdown
Member

Thanks heaps for this @rafrafek, just merged it. 😄

@justinclift

Copy link
Copy Markdown
Member

Awww crap. I was supposed to squash the commits into one before merging this and didn't notice.

Oh well, it's done now. 🤦

@justinclift

Copy link
Copy Markdown
Member

I'm going to manually merge the commits on the master branch.

@justinclift

Copy link
Copy Markdown
Member

I've manually merged the commits in this PR into a single one (b4c46ca) and force pushed it to the master branch. Cleaner commit history that way. 😄

@lucydodo lucydodo mentioned this pull request Feb 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Some links in About window have http protocol rather than https

2 participants