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

Add table tab and add image #2957

Merged
merged 4 commits into from
Jul 5, 2024
Merged

Conversation

barshathakuri
Copy link
Contributor

Changes

  • Add tables and visuals tab
  • Add image in the tab
  • Add pagination in the bottom

This PR doesn't introduce any:

  • temporary files, auto-generated files or secret keys
  • build works
  • eslint issues
  • typescript issues
  • codegen errors
  • console.log meant for debugging
  • typos
  • unwanted comments
  • conflict markers

This PR contains valid:

  • permission checks
  • translations

@barshathakuri barshathakuri marked this pull request as draft June 4, 2024 04:56
@barshathakuri barshathakuri force-pushed the feature/add-data-tables-tab branch 2 times, most recently from 051e309 to d7381f6 Compare June 13, 2024 10:41
app/components/LeftPaneEntries/EntryItem/index.tsx Outdated Show resolved Hide resolved
app/components/LeftPaneEntries/EntryItem/index.tsx Outdated Show resolved Hide resolved
app/components/LeftPaneEntries/index.tsx Outdated Show resolved Hide resolved
app/components/LeftPaneEntries/index.tsx Outdated Show resolved Hide resolved
app/components/LeftPaneEntries/index.tsx Outdated Show resolved Hide resolved
app/views/EntryEdit/index.tsx Outdated Show resolved Hide resolved
app/views/EntryEdit/index.tsx Outdated Show resolved Hide resolved
app/views/EntryEdit/index.tsx Outdated Show resolved Hide resolved
app/views/EntryEdit/queries.ts Outdated Show resolved Hide resolved
@subinasr subinasr force-pushed the feature/add-data-tables-tab branch from d7381f6 to 8c26bfd Compare June 13, 2024 11:03
@barshathakuri barshathakuri force-pushed the feature/add-data-tables-tab branch 5 times, most recently from 6301d76 to 83c05d8 Compare June 17, 2024 12:44
app/components/LeftPaneEntries/index.tsx Outdated Show resolved Hide resolved
app/components/LeftPaneEntries/index.tsx Outdated Show resolved Hide resolved
app/components/entry/ExcerptInput/index.tsx Outdated Show resolved Hide resolved
app/views/EntryEdit/index.tsx Outdated Show resolved Hide resolved
app/views/EntryEdit/index.tsx Outdated Show resolved Hide resolved
@subinasr subinasr force-pushed the feature/add-data-tables-tab branch from b77cfc2 to 65cd265 Compare June 18, 2024 10:45
@subinasr subinasr force-pushed the feature/add-data-tables-tab branch 4 times, most recently from 6e4e5b3 to 5fa40dd Compare June 21, 2024 08:56
@subinasr subinasr marked this pull request as ready for review June 21, 2024 08:56
@subinasr subinasr force-pushed the feature/add-data-tables-tab branch from c6b046d to d313852 Compare June 26, 2024 05:50
app/components/LeftPaneEntries/AssistItem/index.tsx Outdated Show resolved Hide resolved
app/components/LeftPaneEntries/EntryItem/index.tsx Outdated Show resolved Hide resolved
>
<ImagePreview
alt="Preview Image"
src={attachment.filePreview?.url ?? ''}
Copy link
Contributor

Choose a reason for hiding this comment

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

Don't use empty string as default value

app/components/LeftPaneEntries/index.tsx Outdated Show resolved Hide resolved
app/components/entry/ExcerptInput/index.tsx Outdated Show resolved Hide resolved
app/views/EntryEdit/index.tsx Outdated Show resolved Hide resolved
@subinasr subinasr force-pushed the feature/add-data-tables-tab branch from d313852 to 6c02e6c Compare June 26, 2024 09:06
@subinasr subinasr force-pushed the feature/add-data-tables-tab branch 2 times, most recently from 49b505e to 6c1e136 Compare June 27, 2024 06:37
] = useState<boolean>(false);

const leadAttachmentIdsWithEntries = useMemo(() => (
Object.values(entryAttachmentsMap ?? {})
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Object.values(entryAttachmentsMap ?? {})
Object.values(entryAttachmentsMap)

app/components/LeftPaneEntries/index.tsx Outdated Show resolved Hide resolved
@subinasr subinasr force-pushed the feature/add-data-tables-tab branch from 6c1e136 to 35b17c4 Compare July 1, 2024 08:43
@subinasr subinasr requested a review from tnagorra July 2, 2024 08:14
@subinasr subinasr force-pushed the feature/add-data-tables-tab branch from e1994d5 to 0cfa4b7 Compare July 5, 2024 06:45
@frozenhelium frozenhelium merged commit 18de39d into develop Jul 5, 2024
4 of 5 checks passed
@frozenhelium frozenhelium deleted the feature/add-data-tables-tab branch July 5, 2024 07:10
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.

5 participants