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.

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

RC #251 - Peripleo components #252

Merged
merged 109 commits into from
Mar 8, 2024
Merged

Conversation

dleadbetter
Copy link
Collaborator

@dleadbetter dleadbetter commented Feb 6, 2024

This pull request combines the components added in #247, #249, and #250 into several different components. This pull request also creates Storybook stories for all of the new components.

The LocationMarker component was added to the geospatial package.

Screenshot 2024-02-06 at 11 03 52 AM

A new core-data package was created to house all of the Core Data related components. For the time being, this package will be styled using compiled TailwindCSS classes:

Screenshot 2024-02-06 at 11 04 03 AM
Screenshot 2024-02-06 at 11 05 54 AM
Screenshot 2024-02-06 at 11 06 00 AM
Screenshot 2024-02-06 at 11 06 10 AM
Screenshot 2024-02-06 at 11 09 25 AM

import '@performant-software/core-data/build/main.css`

In the future, we should look to allow a plugin to be created to allow the consuming application(s) to override any themes defined in this library.

rsimon and others added 30 commits January 27, 2024 09:06
…-components into feature/rc251_peripleo_components

# Conflicts:
#	packages/geospatial/src/index.js
#	yarn.lock
…tware/react-components into feature/rc251_peripleo_components

# Conflicts:
#	yarn.lock
…-components into feature/rc251_peripleo_components

# Conflicts:
#	packages/controlled-vocabulary/package.json
#	packages/geospatial/package.json
#	packages/semantic-ui/package.json
#	packages/semantic-ui/webpack.config.js
#	packages/shared/package.json
#	packages/user-defined-fields/package.json
#	packages/visualize/package.json
#	react-components.json
@dleadbetter
Copy link
Collaborator Author

dleadbetter commented Mar 1, 2024

I was finally able to get this to work after re-doing the entire build process. I think the changes likely warrant a v2.0 release rather than a v1.2 release?

@jamiefolsom
Copy link
Member

I was finally able to get this to work after re-doing the entire build process. I think the changes likely warrant a v2.0 release rather than a v1.2 release?

Makes sense to me!

Copy link
Collaborator

@rsimon rsimon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow - epic work! :-)

I ran the build + storybook UI from a fresh git clone and everything looks good.

FWIW: also, VSCode is no longer complaining about TS syntax in .js files. So whatever you did, it's great :-)

…-components into feature/rc251_peripleo_components

# Conflicts:
#	packages/controlled-vocabulary/package.json
#	packages/geospatial/package.json
#	packages/semantic-ui/package.json
#	packages/shared/package.json
#	packages/user-defined-fields/package.json
#	packages/visualize/package.json
#	react-components.json
@dleadbetter dleadbetter requested a review from camdendotlol March 4, 2024 16:39
@camdendotlol camdendotlol requested review from ajolipa and removed request for camdendotlol March 5, 2024 15:56
@dleadbetter dleadbetter merged commit b49d491 into master Mar 8, 2024
@dleadbetter dleadbetter deleted the feature/rc251_peripleo_components branch March 8, 2024 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v2.0.0 Issues in v2.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants