-
Notifications
You must be signed in to change notification settings - Fork 2
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
Findby 2 #8
Open
maxf
wants to merge
175
commits into
HarryTrimble:master
Choose a base branch
from
datagovuk:findby-2
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Findby 2 #8
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Tiny readme with the world's most basic instructions
Rather than try to handle weird routes by intercepting urls on the backend, this allows us to call /menu with a query param (it's a GET) called goto, and then redirect to that page. For instance <form action="/menu" method="get"> <radio name="goto" value="/somewhere/togo"> <radio name="goto" value="/somewhere/else"> </form>
Route requests from radio buttons to any page
Adds some templates to work through the suggested harvester workflow. Step 1 is adding a title and a URL, which then submits to a page telling the user we have identified the catalogue and showing a list of organisations from which they can harvest - note there is also a page at /harvest/unknown which asks the user to specify which catalogue they use. The user is then asked how frequently they wish to have the harvester run, then whether they want notifications after each run. And that's it.
Adds some templates for the suggested harvesting workflow
Hosted at /settings, this allows users to: 1. Manage users to invite a new user to the organisation and assign them permissions 2. Manage the contact details for the organisation 3. Change the description and title of the organisation
Adds a settings flow
So that users can see how their harvesters are behaving or when they click on a notification that a harvester failed I've added: * A page that lists top level details about the harvesters that are currently configured * A detail page that shows info on the last run, and what was imported. There is also a view showing what is displayed when a harvester fails (they do).
Adds an overview/detail view of the harvesters
Add hardwired authentication
Fix header
content design
Changed to sign in
Fixes #8
Add dataset list page
Use the same login header, and fix a few links
Add JS code to filter datasets
This should fix #24
Make the settings go straight to the contact details form
Now the user management should be accessible at /team so the dashboard needds to have Team Management that points to here. This fixes #23
Moves the user management to a top level item
Add flow for updating dataset
- multiple file uploads - move file upload towards the end - move notifications before check page - copy fixes
Main features of new workflow:
Fix editing
Latest copy changes, and add financial year
New dashboard design
this is currently breaking edit
Align things better
- remove themes - replace file with links - remove unused templates - rename country to region
Copy changes and flow simplifications
Fix broken link editing
Add indicators if link valid
Add realistic users and data for user-testing sessions
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.