-
Notifications
You must be signed in to change notification settings - Fork 81
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
Feature/#1164 sitelabel #1617
Feature/#1164 sitelabel #1617
Conversation
5b055e3
to
0afce12
Compare
@clash99: Is this PR also intended to merge the project dashboard changes as well? If not, I would prefer that we separate the site label changes from the project dashboard as a completely independent commit instead of branching it off from the feature/project-dashboard branch. Otherwise, I think it would be better to approve and then merge PR #1537 first before merging this PR. |
Some further comments. The site label can't be localized so it will always be in English. I think this should be okay, but just letting everyone know. Also, should this site label be mentioned somehow, somewhere in our documentation (paging @bethschechter and @dpalomino)? If so, then we should create a GitHub issue at the cadasta-docs repo to track this. |
Added Cadasta/cadasta-docs#60 to reflect site label in documentation. |
WIP: fixing dashboard view test WIP: starting map overlay WIP: Add members panel WIP: fixing tests Adding data collection section WIP: more fixes Making header fluid HTML updates Overview and linting More dashboard changes Dashboard and CSS Css changes Swap button and alert Fixed broken test Reworked view for non-project members This is to fill space in right column until request form gets developed Hide project exports button if no content Added hover to cog disable map click for non-project users Remove map overlay when project has locations Map fixes: * Fix map height on location detail page * Fix click handling on location pages for non-project users Adding org admins to members list Updates to overview and dashboards Small tweak and responsive fixes Fixing db queries Removing redundant test code Breaking into subtemplates Changed private icon to label Removed member admin options, updated new user language, private label About projects and about org area and client-side required for description fields Moving comments inside logic Updating schema messaging Minor fix Removing font awesome Adjusted table panel and check color Restructuring welcome text for buttons to align Added "no questionnaire" option CSS for alert Broke org into member and nonmember parts. Improved empty and starter text areas. Eliminate extra space before period Order project members by username Fixed broken tag Changed view for contacts and url, removed plus from button Hid starter text to add team members
Dynamic platform env identifier to base html template CSS adjustments
0afce12
to
847c6b8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! 👍
Thanks @seav for the heads-up. Yes, I agree, I think this is okay. |
Proposed changes in this pull request
When should this PR be merged
Risks
Follow-up actions
Checklist (for reviewing)
General
migration
label if a new migration is added.Functionality
Code
Tests
Security
Documentation