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

Replace username, icon and image path #78

Closed
wants to merge 8 commits into from

Conversation

ZooHigher26
Copy link
Collaborator

@ZooHigher26 ZooHigher26 commented Aug 31, 2024

Description

This PR is related to #53

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

@cbh778899 cbh778899 requested a review from Aisuko August 31, 2024 10:59
@Aisuko Aisuko requested review from cbh778899 and Aisuko and removed request for Aisuko and cbh778899 August 31, 2024 23:36
Copy link
Contributor

@Aisuko Aisuko left a comment

Choose a reason for hiding this comment

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

HI @ZooHigher26 thanks for you PR. You should create PR without using your main branch. Please close this PR and create a new PR with a new branch. The new branch name can be "feat/ui"

@@ -62,7 +62,7 @@ def genRANstring(n):

# CREATE THE SIDEBAR
with st.sidebar:
st.image('img/RMIT_POS3.png', use_column_width=True)
st.image('example/img/log.png', use_column_width=True)
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there any logo that related to your team?. I mean the logo without SkywardAI also be fine.

@Aisuko Aisuko self-assigned this Aug 31, 2024
@Aisuko Aisuko added the python label Aug 31, 2024
@Aisuko Aisuko assigned Micost and Murkeee and unassigned Aisuko Aug 31, 2024
@Aisuko Aisuko requested review from Micost and Murkeee August 31, 2024 23:51
@Aisuko
Copy link
Contributor

Aisuko commented Aug 31, 2024

Hi @Micost @Murkeee feel free to help us review the code

@ZooHigher26 ZooHigher26 closed this Sep 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants