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

update-readme #5

Merged
merged 8 commits into from
Mar 1, 2022
Merged

update-readme #5

merged 8 commits into from
Mar 1, 2022

Conversation

keenangraham
Copy link
Contributor

No description provided.

@forresttanaka forresttanaka merged commit 535a754 into main Mar 1, 2022
@forresttanaka forresttanaka deleted the update-readme branch March 1, 2022 19:34
forresttanaka added a commit that referenced this pull request Mar 8, 2022
@forresttanaka
# This is a combination of 21 commits.
969cfb5 
# This is the 1st commit message:

IGVF-35-cypress-circleci

Try the cypress orb.

# This is the commit message #2:

Revert "IGVF-35-cypress-circleci"

This reverts commit fa1521f.

# This is the commit message #3:

IGVF-35-cypress-circleci

Revert last change and try a different tactic using docker-compose.test.yml.

# This is the commit message #4:

Revert "IGVF-35-cypress-circleci"

This reverts commit 2e7ad9c.

# This is the commit message #5:

IGVF-35-cypress-circleci

Another attempt to run Cypress in CircleCI.

# This is the commit message #6:

IGVF-35-cypress-circleci

Alter previous attempt.

# This is the commit message #7:

IGVF-35-cypress-circleci

Try two docker-compose files; one specifically for cypress.

# This is the commit message #8:

IGVF-35-cypress-circleci

Possibly the wrong command to start Cypress.

# This is the commit message #9:

IGVF-35-cypress-circleci

Try to solve EACCES error for Cypress cache.

# This is the commit message #10:

IGVF-35-cypress-circleci

Try setting environment variable for Cypress in through the docker-compose file.

# This is the commit message #11:

IGVF-35-cypress-circleci

Fix error setting the environment variable from the last fix.

# This is the commit message #12:

IGVF-35-cypress-circleci

Try a different cypress cache directory

# This is the commit message #13:

IGVF-35-cypress-circleci

Everyone seems to recommend a different syntax. Here’s yet another.

# This is the commit message #14:

IGVF-35-cypress-circleci

Downgrading node in case cypress-io/cypress#15025 is correct.

# This is the commit message #15:

IGVF-35-cypress-circleci

Try apt-get for cypress dependencies.

# This is the commit message #16:

Revert "IGVF-35-cypress-circleci"

This reverts commit f7b66a5.

# This is the commit message #17:

IGVF-35-cypress-circleci

Try using the Cypress orb.

# This is the commit message #18:

IGVF-35-cypress-circleci

Start the server before starting the Cypress tests.

# This is the commit message #19:

IGVF-35-cypress-circleci

Correction to last commit. Options can have very subtle effects on required syntax.

# This is the commit message #20:

Updated config.yml
# This is the commit message #21:

IGVF-35-cypress-circleci

Do a build and run of production build.
forresttanaka added a commit that referenced this pull request Mar 8, 2022
@forresttanaka
# This is a combination of 22 commits.
a1fc972 
# This is the 1st commit message:

IGVF-35-cypress-circleci

Try the cypress orb.

# This is the commit message #2:

Revert "IGVF-35-cypress-circleci"

This reverts commit fa1521f.

# This is the commit message #3:

IGVF-35-cypress-circleci

Revert last change and try a different tactic using docker-compose.test.yml.

# This is the commit message #4:

Revert "IGVF-35-cypress-circleci"

This reverts commit 2e7ad9c.

# This is the commit message #5:

IGVF-35-cypress-circleci

Another attempt to run Cypress in CircleCI.

# This is the commit message #6:

IGVF-35-cypress-circleci

Alter previous attempt.

# This is the commit message #7:

IGVF-35-cypress-circleci

Try two docker-compose files; one specifically for cypress.

# This is the commit message #8:

IGVF-35-cypress-circleci

Possibly the wrong command to start Cypress.

# This is the commit message #9:

IGVF-35-cypress-circleci

Try to solve EACCES error for Cypress cache.

# This is the commit message #10:

IGVF-35-cypress-circleci

Try setting environment variable for Cypress in through the docker-compose file.

# This is the commit message #11:

IGVF-35-cypress-circleci

Fix error setting the environment variable from the last fix.

# This is the commit message #12:

IGVF-35-cypress-circleci

Try a different cypress cache directory

# This is the commit message #13:

IGVF-35-cypress-circleci

Everyone seems to recommend a different syntax. Here’s yet another.

# This is the commit message #14:

IGVF-35-cypress-circleci

Downgrading node in case cypress-io/cypress#15025 is correct.

# This is the commit message #15:

IGVF-35-cypress-circleci

Try apt-get for cypress dependencies.

# This is the commit message #16:

Revert "IGVF-35-cypress-circleci"

This reverts commit f7b66a5.

# This is the commit message #17:

IGVF-35-cypress-circleci

Try using the Cypress orb.

# This is the commit message #18:

IGVF-35-cypress-circleci

Start the server before starting the Cypress tests.

# This is the commit message #19:

IGVF-35-cypress-circleci

Correction to last commit. Options can have very subtle effects on required syntax.

# This is the commit message #20:

Updated config.yml
# This is the commit message #21:

IGVF-35-cypress-circleci

Do a build and run of production build.

# This is the commit message #22:

IGVF-35-cypress-circleci

Try anything.
forresttanaka added a commit that referenced this pull request Jul 26, 2022
@forresttanaka
# This is a combination of 5 commits.
a814e9e 
# This is the 1st commit message:

IGVF-218-prettier

Add a prettier checker command and have prettier ignore the cdk directory.

# This is the commit message #2:

IGVF-218-prettier

Add a CLI command to fix files that don’t comply with prettier.

# This is the commit message #3:

IGVF-218-prettier

Try a pre-commit for prettier.

# This is the commit message #4:

IGVF-218-prettier

Test whether I get an error.

# This is the commit message #5:

Another test
forresttanaka added a commit that referenced this pull request Aug 1, 2022
@forresttanaka
# This is a combination of 5 commits.
13463e9 
# This is the 1st commit message:

Commit Log

* Use the collection components common to all other collection pages on the users collection page. We still need to wait for the collection report page ticket to get merged before we finish this.
* Update the individual user page to show more data.

# This is the commit message #2:

IGVF-28-users

Use the rename “lib” directory instead of “libs.” Remove reminder note from app.spec.js. Add table view to users collection page.

# This is the commit message #3:

IGVF-28-users

The user object while not signed in has a lot of missing properties. Make these conditional on the user object page.

# This is the commit message #4:

IGVF-28-users

Add the new testid property in call to CollectionItem on user collection page.

# This is the commit message #5:

IGVF-28-users

Take care of case where a user’s `lab` property doesn’t exist.
forresttanaka added a commit that referenced this pull request Aug 1, 2022
@forresttanaka
# This is a combination of 6 commits.
1bcf416 
# This is the 1st commit message:

Commit Log

* Use the collection components common to all other collection pages on the users collection page. We still need to wait for the collection report page ticket to get merged before we finish this.
* Update the individual user page to show more data.

# This is the commit message #2:

IGVF-28-users

Use the rename “lib” directory instead of “libs.” Remove reminder note from app.spec.js. Add table view to users collection page.

# This is the commit message #3:

IGVF-28-users

The user object while not signed in has a lot of missing properties. Make these conditional on the user object page.

# This is the commit message #4:

IGVF-28-users

Add the new testid property in call to CollectionItem on user collection page.

# This is the commit message #5:

IGVF-28-users

Take care of case where a user’s `lab` property doesn’t exist.

# This is the commit message #6:

IGVF-28-users

Change the order of returned properties in users getServerSideProps() to have more logic.
forresttanaka added a commit that referenced this pull request Jan 13, 2023
@forresttanaka
# This is a combination of 5 commits.
5530e18 
# This is the 1st commit message:

IGVF-343-ui

* Try out some background, data, and border colors.
* New button module with just three types of buttons instead of eight. The main Button component now displays icon buttons as well, and the Button.Icon component no longer exists. Only the access key modules work with it, so much of the rest of the site crashes because of the lack of Button.Icon.
* Intermediate step on the way to converting all buttons to the new paradigm.
* Update all remaining buttons to use the updated Button component.
* Get rid of more old button type usages. I’m sure we still have plenty more old button type usages.
* Finish up button colors. Still having trouble with an infinite loop in Button.Link.
* Update the <Button> API to simplify it and add the new type/size functionality.
* Update all the buttons to use new Tailwind custom classes. Move all form elements to a new components/form-elements directory. I still have to break apart the form.js file into individual component modules and update their clients to refer to them in their new locations.
* Hook up all clients of the form elements to import them from their new location. I still need to get Jest tests working, but will worry about that later. I now need to get `<Select>` objects to use the sm, md, and lg sizes.
* Work out the button heights, hopefully for the last time. Update the dark-mode colors. Add an `<AttachedButton>` wrapper component for buttons to appear attached to each other horizontally.
* Add sizes matching the buttons for the `<Select>` component.

# This is the commit message #2:

IGVF-343-ui

Get rid of last usage of now-unused “info” button type.

# This is the commit message #3:

IGVF-343-ui

Update npm packages.

# This is the commit message #4:

IGVF-343-ui

Add `selected` button type for attached buttons that show selections.

# This is the commit message #5:

IGVF-343-ui

Implement Select and TextField components.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@forresttanaka forresttanaka forresttanaka approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@keenangraham @forresttanaka