Skip to content
This repository has been archived by the owner on May 19, 2023. It is now read-only.

GitHub rudradave1/feature/settingsui #46

Closed

Conversation

rudradave1
Copy link

Added settings screen UI

sumanabhi and others added 3 commits August 29, 2022 13:23
* Adding CODEOWNERS files.

Still needed to add the Default reviewer for the two teams.

* Update CODEOWNERS

* fix the validation issues on GitHub.

* updated owners for all the branches.

* updated the Wrong branch combinations.
Added the PR Templates +  .gitignore
@rudradave1 rudradave1 added the team-loveleen A task assigned to member of team Loveleen label Sep 4, 2022
@rudradave1 rudradave1 linked an issue Sep 4, 2022 that may be closed by this pull request
3 tasks
@rudradave1 rudradave1 marked this pull request as draft September 4, 2022 06:01
@rudradave1 rudradave1 removed a link to an issue Sep 4, 2022
3 tasks
@rudradave1 rudradave1 marked this pull request as ready for review September 4, 2022 06:05
@rudradave1 rudradave1 requested review from loveleenkaurnancy and removed request for hadiyarajesh September 4, 2022 06:05
Copy link
Owner

@hadiyarajesh hadiyarajesh left a comment

Choose a reason for hiding this comment

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

Kindly rebase your branch with @loveleenkaurnancy branch and then PR

@hadiyarajesh
Copy link
Owner

Please change your upstream branch to Loveleen's branch. you don't need to merge to master branch.

@rudradave1 rudradave1 changed the base branch from master to branch_loveleen October 3, 2022 09:48
@hadiyarajesh
Copy link
Owner

@rudradave1 can you rebase your base branch with geet and create a PR for the same?

@hadiyarajesh
Copy link
Owner

I think we don't need this PR as we already have
#56

Am I correct?

@rudradave1 rudradave1 closed this Oct 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
team-loveleen A task assigned to member of team Loveleen
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants