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

Future weather #10 #24

Open
wants to merge 34 commits into
base: main
Choose a base branch
from
Open

Future weather #10 #24

wants to merge 34 commits into from

Conversation

Lunamzelo
Copy link

@Lunamzelo Lunamzelo commented Feb 4, 2023

Contributors, PR Template

Self checklist

  • I have committed my files one by one, on purpose, and for a reason
  • I have titled my PR with ISSUE NUMBER | ISSUE TITLE
  • I have linked my PR to the paired issue with #123
  • I have tested my changes
  • My changes follow the style guide
  • My changes meet the acceptance criteria

Changelist

Briefly explain your PR.

Context

The numbers array is not the actual data, it's just representing how it would be done.

Link to ticket with fixes #123

Questions

Ask any questions you have for your reviewer.

SallyMcGrath and others added 26 commits January 30, 2023 19:49
include jest and rWV
.env files should never be committed, but it's ok to have an example file with fake values to cue developers to update their env
no values in here yet, just examples
added code tags for syntax highlighting
removed git add . advice as it causes chaos
and remove display/copy differentiation
set css variables to be available everywhere
six levels
the most common level will be --theme-spacing---3
removed all unnecessary code
@netlify
Copy link

netlify bot commented Feb 4, 2023

Deploy Preview for rainyplaytime ready!

Name Link
🔨 Latest commit eabce48
🔍 Latest deploy log https://app.netlify.com/sites/rainyplaytime/deploys/63fa01c7f9b61700080138fd
😎 Deploy Preview https://deploy-preview-24--rainyplaytime.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@SallyMcGrath
Copy link
Member

SallyMcGrath commented Feb 4, 2023

fixes #10

Copy link

@LucyMac LucyMac left a comment

Choose a reason for hiding this comment

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

Nice!

delete Emeka's YL

I will recreate it on main
LucyMac
LucyMac approved these changes Feb 25, 2023
@LucyMac LucyMac changed the base branch from master to main March 2, 2023 10:44
Copy link

@LucyMac LucyMac left a comment

Choose a reason for hiding this comment

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

@Lunamzelo I have updated a few things on the main branch, so please could you update your branch future-weather with main? This means:

  • Go to main and pull any changes
  • Go back to your future-weather branch and merge main into your branch
  • Then push
    This should update this PR and show fewer changes.

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.

7 participants