Skip to content
This repository has been archived by the owner on Feb 15, 2024. It is now read-only.

Create ProposalsAndIssues.md #1

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

elsdvlee
Copy link
Collaborator

No description provided.

@GenEars
Copy link

GenEars commented Jun 28, 2023

Thanks @elsdvlee for this nice document 👍

Minor layout issues in the "Patching queries and URI patterns checking" section to solve for better understanding:

  • Adding a line break before "Challenge with the patching approach is two fold:"
  • Adding a line break before "Options for the patching approach, in relation [...]"
  • Making the "The current approach used in the “Designing NORIA” paper [...]" paragraph a part of the "Options for the patching approach, in relation [...]" explanations.

I can update this latter with an additional PR if it is more convenient.

@elsdvlee
Copy link
Collaborator Author

@GenEars Can you check again if it is ok now?

@GenEars
Copy link

GenEars commented Jun 28, 2023

@GenEars Can you check again if it is ok now?

Thanks @elsdvlee
I added two comments to explain better:

Copy link

@GenEars GenEars left a comment

Choose a reason for hiding this comment

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

Nice layout now. Thanks @elsdvlee.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants