Skip to content

Adapt RPB UI for BiblioVino (RPB-37) #52

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

Closed
wants to merge 53 commits into from
Closed

Adapt RPB UI for BiblioVino (RPB-37) #52

wants to merge 53 commits into from

Conversation

fsteeg
Copy link
Member

@fsteeg fsteeg commented Oct 19, 2023

@fsteeg fsteeg self-assigned this Oct 19, 2023
@fsteeg fsteeg changed the base branch from rpb-37-biblioVino to main October 20, 2023 13:33
@fsteeg fsteeg changed the title Set up initial BiblioVino prototype (RPB-37) Adapt RPB UI for BiblioVino (RPB-37) Oct 26, 2023
@fsteeg fsteeg changed the base branch from main to rpb-202-strapiToOpac December 3, 2024 10:32
@fsteeg fsteeg changed the base branch from rpb-202-strapiToOpac to main December 3, 2024 10:32
@acka47
Copy link
Contributor

acka47 commented Mar 24, 2025

We will have to look into how to handle the BiblioVino UI in maintenance modus. Probably, an own repo as clone of this repo would make sense.

@fsteeg fsteeg changed the base branch from main to wip-adrian April 7, 2025 08:43
@fsteeg fsteeg changed the base branch from wip-adrian to main April 7, 2025 08:43
@fsteeg fsteeg changed the base branch from main to wip-adrian April 7, 2025 08:46
@fsteeg fsteeg changed the base branch from wip-adrian to main April 7, 2025 08:46
@fsteeg
Copy link
Member Author

fsteeg commented Apr 8, 2025

As discussed in RPB-272, I've added a BiblioVino-specific README:

https://github.com/hbz/rpb/tree/biblioVino#bibliovino

Requesting review from @acka47.

After approval, we can close this PR (without merging). The diff is linked from the README.

@fsteeg fsteeg requested a review from acka47 April 8, 2025 12:51
@fsteeg fsteeg assigned acka47 and unassigned fsteeg Apr 8, 2025
Copy link
Contributor

@acka47 acka47 left a comment

Choose a reason for hiding this comment

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

+1 LGTM, however, I did not try out whether everything works as documented.

@acka47 acka47 assigned fsteeg and unassigned acka47 Apr 8, 2025
@fsteeg fsteeg closed this Apr 8, 2025
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.

4 participants