Skip to content

Align with 2.5.2. Cleaned up themes and fixtures.#287

Merged
tuj merged 22 commits into
release/3.0.0from
feature/align-with-2.5.2
Sep 8, 2025
Merged

Align with 2.5.2. Cleaned up themes and fixtures.#287
tuj merged 22 commits into
release/3.0.0from
feature/align-with-2.5.2

Conversation

@tuj

@tuj tuj commented Sep 5, 2025

Copy link
Copy Markdown
Contributor

Issue

#249

Description

  • Aligned with v. 2.5.2.
  • Removed themes.
  • Moved themes out of public folder. Added command to copy fixtures assets to public/fixtures.

Checklist

  • My code is covered by test cases.
  • My code passes our test (all our tests).
  • My code passes our static analysis suite.
  • My code passes our continuous integration process.

@tuj tuj self-assigned this Sep 5, 2025
@tuj tuj added the enhancement New feature or request label Sep 5, 2025
@tuj tuj added this to the 3.0.0 milestone Sep 5, 2025
@tuj
tuj requested a review from sinejespersen September 5, 2025 12:57
@tuj tuj mentioned this pull request Sep 5, 2025
36 tasks
}}
>
<RoomInfo>
<RoomInfo className="room-info">

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why add a bunch of classes?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

To make it easier to theme the template.

@sinejespersen sinejespersen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

with a comment...

@tuj
tuj merged commit a4e3aa2 into release/3.0.0 Sep 8, 2025
17 checks passed
@tuj
tuj deleted the feature/align-with-2.5.2 branch September 8, 2025 08:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants