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.

Sign up for GitHub

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

Jump to bottom

Rewrite the Weather to an Edge App #105

Merged
merged 16 commits into from
Aug 15, 2023
Merged

Rewrite the Weather to an Edge App #105

merged 16 commits into from
Aug 15, 2023

Commits on Jul 12, 2023

  1. Move files from src/ to root directory.

    @nicomiguelino
    nicomiguelino committed Jul 12, 2023
    Configuration menu
    Copy the full SHA
    c3a2235 View commit details
    Browse the repository at this point in the history

Commits on Jul 13, 2023

  1. Rewrite the Weather app to an Edge App.

    @nicomiguelino
    nicomiguelino committed Jul 13, 2023
    Configuration menu
    Copy the full SHA
    d4c9dd8 View commit details
    Browse the repository at this point in the history

Commits on Jul 17, 2023

  1. Fix the degrees symbol.

    @nicomiguelino
    nicomiguelino committed Jul 17, 2023
    Configuration menu
    Copy the full SHA
    6e174c6 View commit details
    Browse the repository at this point in the history

Commits on Jul 18, 2023

  1. Modify index.html so that Google Analytics and Sentry integrations 

    will not be enabled when `screenly.settings.disable_analytics` is set to
`true`.
    @nicomiguelino
    nicomiguelino committed Jul 18, 2023
    Configuration menu
    Copy the full SHA
    1c02f3d View commit details
    Browse the repository at this point in the history

  2. Have the script get the OpenWeatherMap API key from settings instead. 

    - Do a quick refactor as well to avoid long lines.
    @nicomiguelino
    nicomiguelino committed Jul 18, 2023
    Configuration menu
    Copy the full SHA
    0d9fdf3 View commit details
    Browse the repository at this point in the history

Commits on Jul 20, 2023

  1. Make use of localStorage to cache weather data.

    @nicomiguelino
    nicomiguelino committed Jul 20, 2023
    Configuration menu
    Copy the full SHA
    dbf61bc View commit details
    Browse the repository at this point in the history

Commits on Jul 21, 2023

  1. Update the script to use IndexedDB instead of localStorage.

    @nicomiguelino
    nicomiguelino committed Jul 21, 2023
    Configuration menu
    Copy the full SHA
    515a5b2 View commit details
    Browse the repository at this point in the history

  2. Fix the manifest file for the Weather Edge App.

    @nicomiguelino
    nicomiguelino committed Jul 21, 2023
    Configuration menu
    Copy the full SHA
    85a9696 View commit details
    Browse the repository at this point in the history

  3. Remove READNE.md as Markdown files are not supported for Edge Apps.

    @nicomiguelino
    nicomiguelino committed Jul 21, 2023
    Configuration menu
    Copy the full SHA
    9857e43 View commit details
    Browse the repository at this point in the history

  4. Change boolean- and secret-typed settings to string.

    @nicomiguelino
    nicomiguelino committed Jul 21, 2023
    Configuration menu
    Copy the full SHA
    4f75801 View commit details
    Browse the repository at this point in the history

  5. Change type from string to text.

    @nicomiguelino
    nicomiguelino committed Jul 21, 2023
    Configuration menu
    Copy the full SHA
    e146465 View commit details
    Browse the repository at this point in the history

Commits on Jul 22, 2023

  1. Modify conditional statement whether to disable analytics in Weather 

    Edge App.
    @nicomiguelino
    nicomiguelino committed Jul 22, 2023
    Configuration menu
    Copy the full SHA
    6ec77b2 View commit details
    Browse the repository at this point in the history

Commits on Aug 3, 2023

  1. Merge branch 'Screenly:master' into 99-weather-app-rewrite

    @nicomiguelino
    nicomiguelino committed Aug 3, 2023
    Configuration menu
    Copy the full SHA
    177f839 View commit details
    Browse the repository at this point in the history

Commits on Aug 10, 2023

  1. Merge branch 'master' into 99-weather-app-rewrite

    @nicomiguelino
    nicomiguelino committed Aug 10, 2023
    Configuration menu
    Copy the full SHA
    6797890 View commit details
    Browse the repository at this point in the history

Commits on Aug 14, 2023

  1. Change data type of sensitive settings to secret instead of text.

    @nicomiguelino
    nicomiguelino committed Aug 14, 2023
    Configuration menu
    Copy the full SHA
    1b1a08e View commit details
    Browse the repository at this point in the history

Commits on Aug 15, 2023

  1. Resolve the most recent code review comments for the weather app. 

    - Host `Moment.js` locally.
- Have the indexed DB be cleared first before caching a new set of data.
- Refactor how the query parameters are declared (for readability).
    @nicomiguelino
    nicomiguelino committed Aug 15, 2023
    Configuration menu
    Copy the full SHA
    d03118e View commit details
    Browse the repository at this point in the history