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

Dark theme contribution #61

Closed

Conversation

PauPau-Cor
Copy link
Contributor

Change resolving #57 by adding new colors.xml file with dark theme qualifier.

line 157 in SenderRecieverService.kt is giving me an unresolvedReference error at compilation, I didn't make any changes to that part of the code or that icon so I left it as is assuming it's something wrong with my compiler.

@ortex
Copy link
Member

ortex commented Feb 11, 2023

hi @PauPau-Cor

Could you please resolve conflicts and rebase on main?

line 157 in SenderRecieverService.kt is giving me an unresolvedReference error at compilation, I didn't make any changes to that part of the code or that icon so I left it as is assuming it's something wrong with my compiler.

Did you try ./gradlew clean build ? 😄

@PauPau-Cor PauPau-Cor force-pushed the dark_theme_contribution branch from 5089750 to 138eaa3 Compare February 16, 2023 03:20
@PauPau-Cor
Copy link
Contributor Author

./gradlew clean build worked, thank you!
I am still new to github, took me a while to figure out resolving the conflicts and rebasing but I think I got it, sorry for the delay.

@ortex
Copy link
Member

ortex commented Feb 16, 2023

I am still new to github, took me a while to figure out resolving the conflicts and rebasing but I think I got it, sorry for the delay.

no worries :)

You merged. Not rebased

easy way to fix it:

  1. sync your fork with upstream https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/syncing-a-fork (just click the button on github UI), I suppose you need discard commits before sync

image

  1. exec git fetch in console

  2. go to your local branch dark_theme_contribution

  3. rebase your branch on origin/main
    I suppose the easiest way to do it is through android studio:

  • open VCS operations -> 7. Branches

image

  • select origin/main branch in Remote Branches, select Rebase 'dark_theme_contribution' onto 'origin/main'

image

  • resolve conflicts

image

  • check that you have only correct changes and force push to origin/dark_theme_contribution

image

@PauPau-Cor
Copy link
Contributor Author

Thank you for the tutorial.
I did every step and it said the reabse was successful but there doesn't seem to be any changes on the remote
I think it's because there aren't any changes to the files so it didn't push anything 😕

@ortex
Copy link
Member

ortex commented Feb 18, 2023

I did every step and it said the reabse was successful but there doesn't seem to be any changes on the remote

looks like you didn't sync your for.
image

After proper sync you should see message: This branch is up to date with roc-streaming/roc-droid:main.

like this:
image

Another way is configure upstream https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/configuring-a-remote-repository-for-a-fork and rebase on upstream/main

@ortex
Copy link
Member

ortex commented Feb 18, 2023

And about PR itself :) Looks good, thanks

I guess text color in IP address input should be fixed in dark theme
image

@gavv
Copy link
Member

gavv commented Feb 18, 2023

And about PR itself :) Looks good, thanks

I guess text color in IP address input should be fixed in dark theme
image

And also tab names

This was referenced Sep 30, 2023
@gavv gavv closed this Oct 12, 2023
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