vinibrsl
Repos
29
Followers
146
Following
230

Elixir is a dynamic, functional language designed for building scalable and maintainable applications

21214
2931

Simple, open-source, lightweight (< 1 KB) and privacy-friendly web analytics alternative to Google Analytics.

13738
687

No dependency Crystal shard to validate, generate and format Brazilian burocracias such as CPF, CNPJ and CEP

21
3

📂 HackerRank SQL track solutions

97
68

🌍 Dataset that shows the Internet affordability by country (a shocking reality!)

📝 A collection of useful .editorconfig templates

26
2

Events

create branch
vinibrsl create branch comparisons
Created at 14 hours ago
issue comment
Interval label flicking when changing metric

hmm it still flickers for me with this patch

flicker-2023-02-01_14.30.50.mp4

Ohhh... thanks for testing! The interval is saved in localStorage when you actually select an interval, and in your case it looks like it's undefined.

I guess we'll need to figure out a better way of doing this. Maybe saving the interval in the query state property so we'll have it available while the HTTP request is not ready yet.

Created at 20 hours ago
pull request opened
Interval label flicking when changing metric

This commit fixes a bug where the interval label flickers when changing the dashboard metric. This was caused because changing the metric clears the graphData state property until the HTTP request is finished.

This change fixes it by falling back interval to the value stored in localStorage when graphData is falsy.

Bug preview (see "Days")

preview.webm

Created at 22 hours ago
create branch
vinibrsl create branch fix-interval-label-flicker
Created at 22 hours ago
delete branch
vinibrsl delete branch reject-short-error-feedback
Created at 1 day ago

Prevent error reports from being sent with short feedback (#2635)

Created at 1 day ago
pull request closed
Prevent error reports from being sent with short feedback

Changes

This PR makes sure no e-mail is sent when the users submit feedback with less than 5 characters.

Tests

  • [x] Automated tests have been added
  • [ ] This PR does not require tests

Changelog

  • [ ] Entry has been added to changelog
  • [x] This PR does not make a user-facing change

Documentation

  • [ ] Docs have been updated
  • [x] This change does not need a documentation update

Dark mode

  • [ ] The UI has been tested both in dark and light mode
  • [x] This PR does not change the UI
Created at 1 day ago
issue comment
Add city name to imported_locations table

Please don't merge this pull request after review 💙 I'll do that by myself so I can coordinate the deployment.

Created at 1 day ago
pull request opened
Add city name to imported_locations table

See also #2608

Created at 1 day ago
create branch
vinibrsl create branch migration--import-cities-from-ga
Created at 1 day ago

Fix returning more pageviews with a visit property filter (#2612)

  • fix subquery for sessions in base_event_query/2

As the 'sessions' table is using the CollapsingMergeTree engine, we have to select session_id's distinctively. Otherwise we will get multiple rows (with sign -1 and 1) as long as the background merge hasn't happened.

  • update changelog

  • use GROUP BY instead of SELECT DISTINCT

  • remove comma

parse date labels by local timezone instead of UTC (#2601)

Some changes to the "locked site" copy (#2615)

Some changes to be more consistent with the emails we send. Also "valid" subscription rather than "active" subscription fits better for the different cases where this screen is shown

Revert "parse date labels by local timezone instead of UTC (#2601)" (#2619)

This reverts commit 7a7076ed209afa0549cbd976c7d13bca481b703c.

Show user feedback form on server errors (#2617)

  • Move Endpoint errors setup to common config

  • Implement naive Sentry link resolver

  • Implement error report e-mail

  • Delete static sentry script

  • Implement user feedback form on server errors

  • Re-arrange pipe

  • Use Sentry.Config.dsn() where applicable

  • Fix typo

  • Use Map.replace/3

Changes to the site settings danger zone (#2629)

  • Changes to the site settings danger zone

The idea was to:

  1. Show a transfer site ownership message to owners of the site only. This will make ownership transfer feature more discoverable. Will also help people to transfer ownership of a site rather than deleting the whole thing when that makes more sense
  2. Add a note that site deletion is a delayed operation
  3. Make some other tiny copy changes

What did I break? :)

  • Fix markup so the long text wraps sooner

  • Fix indent


Co-authored-by: Adam Rutkowski hq@mtod.org

Add city geolocation data to seeds (#2626)

Delay stats deletions (#2632)

  • Implement Site removal transaction

  • Implement Stats removal Oban worker

  • Configure site removal queue

  • Call Site.Removal.run() instead of Purge.delete_site!

  • Test site/stats removal

  • Remove FIXME - filed a ticket

  • Over-communicate lenghty deletion process to the users

Remove the ability to collapse the main graph + transition bug fix (#2627)

  • Remove the ability to collpase the top graph

This commit removes the ability to collapse the top graph. The graph collapsed whenever metric was falsy. I removed all related code to that. Metric now defaults to visitors.

We want to add new items to top stats, and this commit will make it easier to change it. Also, there's currently a bug where top stats is randomly collapsing, which should be fixed by this commit.

  • Refactor graph and top stats loading state

The graph loading state shows and hides the graph conditionally depending on whether the data is loaded, loading or refreshing.

The current code is a bit difficult to read because its big conditionals. This commit refactors the loading state making it easier to read.

This commit also fixes a bug where the graph wasn't fading out when changing metrics.

Save city name when importing from GA

This commit adds city data to imported records from Google Analytics. The current implementation sets city to 0 because GA does not use the GeoNames database.

Google Analytics Reporting API uses Geographical IDs to identify cities and countries. Plausible uses GeoNames and I couldn't find databases corelating the two.

Fortunately, GA also returns the city name and this commit uses the city name when querying imported data. I tried deleting the imported_locations.city column from ClickHouse, but this is primary key and can't be deleted. I also couldn't make city flags work, because the country code is from the GeoName, and we don't have it for imported data.

Change Map.merge call to Map.put

Co-authored-by: Adam Rutkowski hq@mtod.org

Revert Logger calls from ObanErrorReporter

Make city name condition clearer

Created at 1 day ago

Fix returning more pageviews with a visit property filter (#2612)

  • fix subquery for sessions in base_event_query/2

As the 'sessions' table is using the CollapsingMergeTree engine, we have to select session_id's distinctively. Otherwise we will get multiple rows (with sign -1 and 1) as long as the background merge hasn't happened.

  • update changelog

  • use GROUP BY instead of SELECT DISTINCT

  • remove comma

parse date labels by local timezone instead of UTC (#2601)

Some changes to the "locked site" copy (#2615)

Some changes to be more consistent with the emails we send. Also "valid" subscription rather than "active" subscription fits better for the different cases where this screen is shown

Revert "parse date labels by local timezone instead of UTC (#2601)" (#2619)

This reverts commit 7a7076ed209afa0549cbd976c7d13bca481b703c.

Show user feedback form on server errors (#2617)

  • Move Endpoint errors setup to common config

  • Implement naive Sentry link resolver

  • Implement error report e-mail

  • Delete static sentry script

  • Implement user feedback form on server errors

  • Re-arrange pipe

  • Use Sentry.Config.dsn() where applicable

  • Fix typo

  • Use Map.replace/3

Changes to the site settings danger zone (#2629)

  • Changes to the site settings danger zone

The idea was to:

  1. Show a transfer site ownership message to owners of the site only. This will make ownership transfer feature more discoverable. Will also help people to transfer ownership of a site rather than deleting the whole thing when that makes more sense
  2. Add a note that site deletion is a delayed operation
  3. Make some other tiny copy changes

What did I break? :)

  • Fix markup so the long text wraps sooner

  • Fix indent


Co-authored-by: Adam Rutkowski hq@mtod.org

Add city geolocation data to seeds (#2626)

Delay stats deletions (#2632)

  • Implement Site removal transaction

  • Implement Stats removal Oban worker

  • Configure site removal queue

  • Call Site.Removal.run() instead of Purge.delete_site!

  • Test site/stats removal

  • Remove FIXME - filed a ticket

  • Over-communicate lenghty deletion process to the users

Remove the ability to collapse the main graph + transition bug fix (#2627)

  • Remove the ability to collpase the top graph

This commit removes the ability to collapse the top graph. The graph collapsed whenever metric was falsy. I removed all related code to that. Metric now defaults to visitors.

We want to add new items to top stats, and this commit will make it easier to change it. Also, there's currently a bug where top stats is randomly collapsing, which should be fixed by this commit.

  • Refactor graph and top stats loading state

The graph loading state shows and hides the graph conditionally depending on whether the data is loaded, loading or refreshing.

The current code is a bit difficult to read because its big conditionals. This commit refactors the loading state making it easier to read.

This commit also fixes a bug where the graph wasn't fading out when changing metrics.

Revert Logger calls from ObanErrorReporter

Make city name condition clearer

Created at 1 day ago

Change Map.merge call to Map.put

Co-authored-by: Adam Rutkowski hq@mtod.org

Created at 1 day ago
delete branch
vinibrsl delete branch remove-hide-top-graph
Created at 1 day ago

Remove the ability to collapse the main graph + transition bug fix (#2627)

  • Remove the ability to collpase the top graph

This commit removes the ability to collapse the top graph. The graph collapsed whenever metric was falsy. I removed all related code to that. Metric now defaults to visitors.

We want to add new items to top stats, and this commit will make it easier to change it. Also, there's currently a bug where top stats is randomly collapsing, which should be fixed by this commit.

  • Refactor graph and top stats loading state

The graph loading state shows and hides the graph conditionally depending on whether the data is loaded, loading or refreshing.

The current code is a bit difficult to read because its big conditionals. This commit refactors the loading state making it easier to read.

This commit also fixes a bug where the graph wasn't fading out when changing metrics.

Created at 1 day ago