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

Tech Talk page #147

Closed
wants to merge 5 commits into from
Closed

Conversation

imrishabh18
Copy link

@imrishabh18 imrishabh18 commented Sep 22, 2020

Description

I have made the carousel with proper props and state. Some styling is still left as I was not getting the proper colors. Will be helpful if you can send me those as well as the video links. For the time being, I have used a random photo.

Fixes #122

Type of Change:

  • Code
  • User Interface

Checklist:

  • [✅ ] My PR follows the style guidelines of this project
  • [✅] I have performed a self-review of my own code or materials
  • I have commented my code or provided relevant documentation, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • Any dependent changes have been merged

@imrishabh18
Copy link
Author

@nandini45 I have made a new PR because that branch had some messed up changes. Here, I have removed the unsplash images link too. Even after that It has some build error.

@Akanksha1212 Akanksha1212 added the Status: Needs Review PR needs an additional review or a maintainer's review. label Oct 3, 2020
Copy link
Contributor

@annabauza annabauza left a comment

Choose a reason for hiding this comment

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

@imrishabh18 this is not react native PR. Please update.

@annabauza annabauza added Status: Changes Requested Changes are required to be done by the PR author. and removed Status: Needs Review PR needs an additional review or a maintainer's review. labels Oct 24, 2020
@nandini45
Copy link
Member

@imrishabh18 Can you make the requested changes?

Copy link
Member

@isabelcosta isabelcosta left a comment

Choose a reason for hiding this comment

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

@imrishabh18 in GitHub if you refer to issues in the PR description like: Fixes #122 GitHub will close the issue once the PR is merged, so I changed that in your PR. Now you can follow this practice in future PRs :)

@nandini45
Copy link
Member

@imrishabh18 thank u for the work. Because of your unavailability i will close the pr. Reopen this isse

@nandini45 nandini45 closed this Dec 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Changes Requested Changes are required to be done by the PR author.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

coding: develop the tech talk section in events page
5 participants