Skip to content

Fully qualify RSS URLs #18

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

Merged
merged 1 commit into from
Jan 5, 2024
Merged

Conversation

interrobot
Copy link

Change RSS url from /index.xml to https://sqlitebrowser.org/ globally. Resolves issue #9.

@interrobot
Copy link
Author

That should read... from /index.xml to https://sqlitebrowser.org/index.xml, my mistake.

@justinclift
Copy link
Member

Thanks. Lets try it out, and see how it goes. 😄

The website automatically updates within about 10 mins from updates to this repo, so we should be able to see the result live pretty soon. 😄

@justinclift justinclift merged commit adea5ef into sqlitebrowser:master Jan 5, 2024
@interrobot
Copy link
Author

interrobot commented Jan 6, 2024

Thanks for the merge. I didn't include the exported files (html) in the pull request, as there were a multitude of changes beyond my own... took the cautious route. I believe this is why the change hasn't propagated yet. I did spot check various pages the exported site to verify the RSS update, and it all looked good. The update should drop next time the compiled output is generated into the repo.

@justinclift
Copy link
Member

Ahhh. Want to do a follow up PR with the generated files as well? 😄

@interrobot
Copy link
Author

Yes, absolutely. I'd just want someone close to the project to eyeball the diff. I'll get the pr together today.

@interrobot interrobot deleted the links-fixes branch January 7, 2024 19:41
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.

3 participants