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

feat(www): add unbird feedback form component #10450

Merged
merged 7 commits into from
Dec 14, 2018

Conversation

amberleyromo
Copy link
Contributor

add unbird feedback form component

@amberleyromo amberleyromo requested a review from a team as a code owner December 13, 2018 03:45
Copy link
Contributor

@DSchau DSchau left a comment

Choose a reason for hiding this comment

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

Looks good, left two comments!

www/src/components/unbird.js Show resolved Hide resolved
www/src/components/unbird.js Outdated Show resolved Hide resolved
DSchau and others added 2 commits December 13, 2018 10:41
Co-Authored-By: amberleyromo <amberleyromo@users.noreply.github.com>
Copy link
Contributor

@greglobinski greglobinski left a comment

Choose a reason for hiding this comment

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

@amberleyromo That was my first code review ever. So I'm not sure if that is what you asked for. I wasn't able to suggest you any changes (besides these organisational) without writing real code so all code is already written, if you want use it. :) If it's not what you expected we have to talk about what 'code review' is, so in the future I will be more accurate :)

here is the link to the code in my fork
https://github.com/greglobinski/gatsby/blob/starters/add-unbird-form/www/src/components/unbird.js

www/src/components/unbird.js Outdated Show resolved Hide resolved
www/src/components/unbird.js Outdated Show resolved Hide resolved
www/src/components/unbird.js Show resolved Hide resolved
www/src/components/unbird.js Outdated Show resolved Hide resolved
www/src/components/unbird.js Outdated Show resolved Hide resolved
www/src/components/unbird.js Show resolved Hide resolved
www/src/components/unbird.js Outdated Show resolved Hide resolved
www/src/components/unbird.js Show resolved Hide resolved
www/src/components/unbird.js Show resolved Hide resolved
www/src/components/unbird.js Outdated Show resolved Hide resolved
@greglobinski
Copy link
Contributor

screenshot from 2018-12-14 16-07-50
screenshot from 2018-12-14 16-07-34

Copy link
Contributor

@DSchau DSchau left a comment

Choose a reason for hiding this comment

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

This looks solid, and definitely quite a bit preferable to embedding the widget.

Looks good to me!

@amberleyromo amberleyromo merged commit e9e101c into gatsbyjs:master Dec 14, 2018
wconnorwalsh added a commit to wconnorwalsh/gatsby that referenced this pull request Dec 15, 2018
* master:
  feat(www): add unbird feedback component to starter lib (gatsbyjs#10450)
  fix(blog): youfit case study typofix
  Doc improvements to Visual testing with Storybook guide (gatsbyjs#10436)
  fix(gatsby-plugin-offline): prevent incorrect revisioning of static file by workbox (gatsbyjs#10416)
  fix(starters): ttag repo link
  fix typo in pull request template (gatsbyjs#10454)
  fix(www) Fix query for plugin links always ?=undefined (gatsbyjs#10453)
  chore(release): Publish
  fix(gatsby): fix extracting StaticQuery nested in shorthand fragment (gatsbyjs#10443)
m-allanson added a commit to lipis/gatsby that referenced this pull request Dec 17, 2018
* master: (1037 commits)
  Update starters.yml (gatsbyjs#10505)
  chore(release): Publish
  fix(graphql-skip-limit): fix hasNextPage (gatsbyjs#10504)
  chore: use cjs instead of esm for consistency (gatsbyjs#10494)
  feat(gatsby-remark-copy-linked-files): add support for video elements with `src` attribute (gatsbyjs#10395)
  typofix (gatsbyjs#10488)
  Add kobit.in to showcase (gatsbyjs#10496)
  fix(docs): window.reload => window.location.reload (gatsbyjs#10459)
  feat(www): add unbird feedback component to starter lib (gatsbyjs#10450)
  fix(blog): youfit case study typofix
  Doc improvements to Visual testing with Storybook guide (gatsbyjs#10436)
  fix(gatsby-plugin-offline): prevent incorrect revisioning of static file by workbox (gatsbyjs#10416)
  fix(starters): ttag repo link
  fix typo in pull request template (gatsbyjs#10454)
  fix(www) Fix query for plugin links always ?=undefined (gatsbyjs#10453)
  chore(release): Publish
  fix(gatsby): fix extracting StaticQuery nested in shorthand fragment (gatsbyjs#10443)
  fix(www): avoid querying for no-cache=1 (gatsbyjs#10389)
  fix(gatsby-image): update typescript definitions - properly mark fields as optional (gatsbyjs#10419)
  refactor(gatsby): improve EnsureResources (gatsbyjs#10224)
  ...
gpetrioli pushed a commit to gpetrioli/gatsby that referenced this pull request Jan 22, 2019
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