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

Header opens a new page with no steps #599

Closed
wants to merge 1 commit into from
Closed

Header opens a new page with no steps #599

wants to merge 1 commit into from

Conversation

aashna27
Copy link

@aashna27 aashna27 commented Jan 3, 2019

Fixes #568

Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!

  • tests pass -- look for a green checkbox ✔️ a few minutes after opening your PR -- or run tests locally with rake test
  • code is in uniquely-named feature branch and has no merge conflicts
  • PR is descriptively titled
  • ask @publiclab/reviewers for help, in a comment below

I have left the button part believing that @Mridul97 would be interested to go for it.

@aashna27 aashna27 requested a review from jywarren January 3, 2019 18:31
@Mridul97
Copy link

Mridul97 commented Jan 3, 2019

Great! Looks good! I will open a PR soon with a button in the footer to start a new sequence on the same page. Thanks!

@jywarren
Copy link
Member

jywarren commented Jan 3, 2019

Oops, conflicts!

@aashna27
Copy link
Author

aashna27 commented Jan 3, 2019

Oops, conflicts!

yepp new commit 😄 I ll make a new one then

@aashna27
Copy link
Author

aashna27 commented Jan 3, 2019

is it okay if i dont push the dist files next time ?
cause with the new commits it gives conflicts? @jywarren

@jywarren
Copy link
Member

jywarren commented Jan 3, 2019

Maybe we need to come up with a new strategy for this, because we do need to build each time, but indeed it makes rebasing/merging a real pain! I wonder if there is a conventional way to manage this...? could bear searching around a bit...

@aashna27 aashna27 closed this Jan 4, 2019
@aashna27 aashna27 deleted the Header-reloads-page branch January 4, 2019 10:30
@jywarren
Copy link
Member

jywarren commented Jan 4, 2019

OK, discussion ongoing in #601 !

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