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

Move all demo pages to a demos/ subfolder #3302

Merged
merged 6 commits into from
Jan 15, 2024
Merged

Move all demo pages to a demos/ subfolder #3302

merged 6 commits into from
Jan 15, 2024

Conversation

lminiero
Copy link
Member

@lminiero lminiero commented Dec 6, 2023

After updating demos and docs to use Bootstrap 5.x in #3300, I reorganized the html folder a bit, by moving all demos to a demos subfolder: the purpose was to cleanup the web folder a bit, like when we moved all the code to src.

As part of this restructuring, I also renamed most of the demo pages too: many of them still had a test in it that was unneeded, so I got rid of it. Besides, I added a link to the related plugin's documentation on top of every demo: this will hopefully make people more aware of which plugin is powering each demo, and convince them to checkout the related docs (which not everyone does, unfortunately) if they want to learn more.

@lminiero lminiero added the multistream Related to Janus 1.x label Dec 6, 2023
@lminiero lminiero changed the title Move all demo pages to a demos/ subfolderRename demos Move all demo pages to a demos/ subfolder Dec 6, 2023
Copy link
Contributor

@bkmgit bkmgit left a comment

Choose a reason for hiding this comment

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

Maybe change https://github.com/meetecho/janus-gateway/pull/3302/files#diff-0462e381b2fb3286568215681c8983490a37ac9ae0f0c5ee304df7fa6426d4afR24 to

demosdir = $(datadir)/janus/

so that installed demos are in $(datadir)/janus/demos and not $(datadir)/janus/demos/demos

@lminiero
Copy link
Member Author

That wouldn't work, because it would put all html files nt in demos to be put in the janus parent folder, which would be wrong. But I agree the double demos isn't a good look: maybe we can do $(datadir)/janus/html, which would make more sense since html is how it's called in the repo too (which would make it $(datadir)/janus/html/demos for demos, as expected).

@bkmgit
Copy link
Contributor

bkmgit commented Dec 18, 2023

That seems ok, thanks.

@lminiero lminiero merged commit 5a10a6c into master Jan 15, 2024
8 checks passed
@lminiero lminiero deleted the rename-demos branch January 15, 2024 09:50
Rabenherz112 added a commit to Rabenherz112/awesome-selfhosted-data that referenced this pull request Jan 16, 2024
nodiscc pushed a commit to awesome-selfhosted/awesome-selfhosted-data that referenced this pull request Jan 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
multistream Related to Janus 1.x
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants