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

feat(pages): new architecture page #9

Merged
merged 3 commits into from
Jul 10, 2023
Merged

Conversation

wdavidw
Copy link
Collaborator

@wdavidw wdavidw commented Jun 17, 2023

No description provided.

@wdavidw wdavidw force-pushed the feat/david-page-architecture branch from 7d78da7 to 98be99a Compare June 19, 2023 13:37
@mehdibn
Copy link
Contributor

mehdibn commented Jun 20, 2023

@wdavidw Thank you for this redesign and new implementation of the website, much appreciated !

As noted by @rpignolet, i think that extra components (and especially airflow) should not be part of the reference architecture.

Also, I don't see our proposal (the old French content) about architecture, features, etc. What do you think ?

Thank you again !

@wdavidw
Copy link
Collaborator Author

wdavidw commented Jun 20, 2023

@rpignolet wrote "Airflow is not a core component currently, we should not have it here.", he didn't mention removing it. My personal take is that extra shall be covered. I have never been in agreement with the notion of core or extra, the segregation is counter-productive. Instead, what we have are different levels of maturity, some levels not being supported because too immature, and eventually a playground. In the example of Airflow, the component is too essential to be considered optional or part of a playground. For the old French content, I would not recommend starting from there. I tried to preserve some of it on the home page and would likely rewrite it if I have time.

@wdavidw
Copy link
Collaborator Author

wdavidw commented Jun 20, 2023

To complement, the component list is not exhaustive, additional services shall enrich the page, including what is in tdp-collection-prerequisites and tdp-collection-extras.

@mehdibn
Copy link
Contributor

mehdibn commented Jun 20, 2023

@wdavidw thank you again ! i suggest discussing this point during our next meeting ..

@PACordonnier PACordonnier self-requested a review June 30, 2023 12:13
@wdavidw wdavidw merged commit b60548f into master Jul 10, 2023
1 check failed
@wdavidw wdavidw deleted the feat/david-page-architecture branch July 10, 2023 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants