-
Notifications
You must be signed in to change notification settings - Fork 7
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
Add a new readme #37
Add a new readme #37
Conversation
4a32eb5
to
fb48b0c
Compare
d133c76
to
d9180c8
Compare
@OtterWays any chance this PR will be review in the next months? |
Sorry! I'll do the review within the week |
No problem :) |
7dd765e
to
9c5f844
Compare
@OtterWays ready to be review |
9c5f844
to
d1647da
Compare
Rework readme and add images and links around the panoramax project
d1647da
to
192a04f
Compare
What is blocking the merge of this PR? |
IMO nothing, just @OtterWays has not take time to review Result is available here https://github.com/Jean-BaptisteC/panoramax-mobile-app/tree/readme |
If she's not available, I think we can move on with this (I'd only only show the download logo from where it's currently possible, else it could mislead some users) |
it seems the CI is failing:
It seems that we should upgrade AGP or downgrade the compileSdk version (I don't know the implications of those choices 😅 ) |
Upgrade sdk require to doing a code migration SDK 35 is new the new sdk to add support for Android 15 |
@Jean-BaptisteC do you think you could give me the right to approve and merge PR on this repository? I think lots of incoming changes are stalled and it would be easier to merge those before migrating to gitlab |
I have no right on this repository 😕 |
Hello, Indeed, I can grant you the right to merge code, but I need a confirmation that your are a member of Panoramax Team. |
Panoramax Team is defined here |
This PR upgrade readme:
Todo: