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

Added a mechanism to configure the HTML page title from the server #292

Merged
merged 2 commits into from
Mar 24, 2021

Conversation

adieyal
Copy link
Contributor

@adieyal adieyal commented Mar 23, 2021

Description

Created a general.js setup page and move the analytics configuration there as well.

Related Issue

#286

How to test it locally

added "page_title": "This is my title" into the profile configuration. You should see the change in the title of the page.

Screenshots

Changelog

Added

general.js to store page specific setup

Updated

Removed

Checklist

  • 🚀 is the code ready to be merged and go live?
  • 🛠 does it work (build) locally
  • 👩‍🎨 does the design matches the Demo

Pull Request

  • 📰 good title
  • 📝good description
  • 🔖 issue linked
  • 📖 changelog filled out
  • commit messages are meaningful

Code Quality

  • 🚧 no commented out code
  • 🖨 no unnecessary logging
  • 🎱 no magic numbers

Testing

  • ✅ added (appropriate) unit tests
  • 💢 edge cases in tests were considered
  • ✅ ran tests locally & are passing

…nfiguration

Created a general.js setup page and move the analytics configuration there as well.
@netlify
Copy link

netlify bot commented Mar 23, 2021

Deploy preview for wazimap-staging ready!

Built with commit 84167ca

https://deploy-preview-292--wazimap-staging.netlify.app

@milafrerichs milafrerichs merged commit 46961b7 into staging Mar 24, 2021
@adieyal adieyal deleted the feature/update-page-title branch March 24, 2021 08:06
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.

2 participants