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

Add git-hub handle for Tien Yuan in heart #7001

Closed
wants to merge 0 commits into from

Conversation

troyfreed
Copy link
Member

Fixes #6941

What changes did you make?

-added a variable "github-handle" to Tien Yuan's name in project heart

Why did you make the changes (we will use this info to test)?

  • So there can be a space when looking at this project that shows the github-handles of group members

Screenshots of Proposed Changes Of The Website (if any, please do not screen shot code changes)

There are no changes to the website because "github-handle" has no value yet.

@HackforLABot HackforLABot added this to PR Needs review (Automated Column, do not place items here manually) in Project Board Jun 13, 2024
Copy link

Want to review this pull request? Take a look at this documentation for a step by step guide!


From your project repository, check out a new branch and test the changes.

git checkout -b troyfreed-gh-pages gh-pages
git pull https://github.com/troyfreed/website.git gh-pages

@github-actions github-actions bot added good first issue Good for newcomers role: front end Tasks for front end developers role: back end/devOps Tasks for back-end developers P-Feature: Project Info and Page A project's detail page (e.g. https://www.hackforla.org/projects/100-automations) size: 0.25pt Can be done in 0.5 to 1.5 hours labels Jun 13, 2024
@@ -52,6 +52,7 @@ leadership:
github: https://github.com/abregorivas
picture: https://avatars2.githubusercontent.com/u/7821047?s=400&u=9128a4d0a4d1ff33bdebae02bcdaac7ff7e7432f&v=4
- name: Tien Yuan
github-handle:
Copy link
Member

Choose a reason for hiding this comment

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

This is correct

Project Board automation moved this from PR Needs review (Automated Column, do not place items here manually) to test-approved-by-reviewer (Automated Column, do not place items here manually) Jun 13, 2024
@del9ra del9ra self-requested a review June 14, 2024 01:10
@del9ra
Copy link
Member

del9ra commented Jun 14, 2024

ETA: 06/13/24
Availability: 1-5pm monday-friday

Copy link
Member

@del9ra del9ra left a comment

Choose a reason for hiding this comment

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

Hey @troyfreed,

Project Board automation moved this from test-approved-by-reviewer (Automated Column, do not place items here manually) to test-pending-approval (Automated Column, do not place items here manually) Jun 14, 2024
@troyfreed
Copy link
Member Author

troyfreed commented Jun 18, 2024 via email

@ExperimentsInHonesty ExperimentsInHonesty added role: back end and removed role: back end/devOps Tasks for back-end developers labels Jun 18, 2024
@del9ra
Copy link
Member

del9ra commented Jun 18, 2024

I just fixed the issue, let me know if there is anything else I need to change.

On Thu, Jun 13, 2024 at 8:28 PM Delgira Dordzhieva @.> wrote: @.* requested changes on this pull request. Hey @troyfreed https://github.com/troyfreed, - Your branch title should be a brief description of the issue that will make sense at a glance to someone unfamiliar with the issue. Refer to the guide https://github.com/hackforla/website/blob/gh-pages/CONTRIBUTING.md#27b-working-on-an-issue-2-create-a-new-branch-where-you-will-work-on-your-issue - Just a small notice, check the spelling of “git-hub handle”. I think “github-handle” would be more appropriate - Issue number was listed - The PR request clearly states what and why was updated - The code changes were applied correctly Thanks! Once you’ve made these updates, I’ll be ready to approve it. — Reply to this email directly, view it on GitHub <#7001 (review)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AXC3V32Z4IIC2RYZZEOSU5DZHJBMZAVCNFSM6AAAAABJJASLLCVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDCMJXGI2TSNJYGY . You are receiving this because you were mentioned.Message ID: @.***>

You should create a branch like this: git checkout -b update-give-link-2093:
Screenshot 2024-06-18 at 10 49 49
Refer to this guide: https://github.com/hackforla/website/blob/gh-pages/CONTRIBUTING.md#27b-working-on-an-issue-2-create-a-new-branch-where-you-will-work-on-your-issue

@troyfreed
Copy link
Member Author

troyfreed commented Jun 20, 2024 via email

@jphamtv
Copy link
Member

jphamtv commented Jun 20, 2024

Hi @troyfreed,

It looks like you're using the main branch (gh-pages) for this pull request, which isn't permitted. Please create a feature branch for your changes and submit the pull request from that branch. I suggest closing this PR and starting fresh with a feature branch.

Please refer to the Working an issue section in the contributing guide. It applies whether you are using the terminal or GUI in VS Code.

If you have any questions or need assistance, please attend one of the dev meetings or reach out to the #hfla-site channel on Slack. Thanks!

@troyfreed troyfreed marked this pull request as draft June 24, 2024 23:00
@troyfreed
Copy link
Member Author

troyfreed commented Jun 24, 2024 via email

@Thinking-Panda
Copy link
Member

@troyfreed Issue #6941 has not been worked on and merged yet. It is still open. As requested in earlier comments, please close this PR, and then follow to instructions in CONTIBUTING.md-2.7.b to start working on the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers P-Feature: Project Info and Page A project's detail page (e.g. https://www.hackforla.org/projects/100-automations) role: back end role: front end Tasks for front end developers size: 0.25pt Can be done in 0.5 to 1.5 hours
Projects
Archived in project
Project Board
  
test-pending-approval (Automated Colu...
Development

Successfully merging this pull request may close these issues.

Add github-handle element for Tien Yuan overview in project heart
6 participants