-
-
Notifications
You must be signed in to change notification settings - Fork 660
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
Code changes (04/08) #1418
Code changes (04/08) #1418
Conversation
Related to #612
Last time it was period-first (shown by period, then filter for search), now it is filter-first
It is unused, and I think it can be substituted with the layer system
Also make updateViewFromHash work again
…into feat/decreases_lineCanvas_rerender # Conflicts: # web/_js/main/main.js # web/_js/main/view.js
It has been covered in literally the next lines
✅ Deploy Preview for place-atlas-2023 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Qodana for JSIt seems all right 👌 No new problems were found according to the checks applied 💡 Qodana analysis was run in the pull request mode: only the changed files were checked View the detailed Qodana reportTo be able to view the detailed Qodana report, you can either:
To get - name: 'Qodana Scan'
uses: JetBrains/[email protected]
with:
upload-result: true Contact Qodana teamContact us at [email protected]
|
# Conflicts: # web/_js/main/main.js # web/_js/main/view.js
Okay, merging in no earlier than 2 August, 00:00 GMT |
Nope, something went wrong, pushed some fixes. Will merge in 04/08. |
Revert "Merge pull request #1418 from placeAtlas/staging/code"
This are significant code changes for this period of time. Check the commits for more information.